Reverting due to memory waterfall failures.

base/values.cc

Index: base/values.cc
===================================================================
--- base/values.cc	(revision 138049)
+++ base/values.cc	(working copy)
@@ -298,32 +298,33 @@
 
 ///////////////////// BinaryValue ////////////////////
 
-BinaryValue::BinaryValue()
-    : Value(TYPE_BINARY),
-      buffer_(NULL),
-      size_(0) {
+BinaryValue::~BinaryValue() {
+  DCHECK(buffer_);
+  if (buffer_)
+    delete[] buffer_;
 }
 
-BinaryValue::BinaryValue(scoped_ptr<char> buffer, size_t size)
-    : Value(TYPE_BINARY),
-      buffer_(buffer.release()),
-      size_(size) {
-}
+// static
+BinaryValue* BinaryValue::Create(char* buffer, size_t size) {
+  if (!buffer)
+    return NULL;
 
-BinaryValue::~BinaryValue() {
+  return new BinaryValue(buffer, size);
 }
 
 // static
 BinaryValue* BinaryValue::CreateWithCopiedBuffer(const char* buffer,
                                                  size_t size) {
+  if (!buffer)
+    return NULL;
+
   char* buffer_copy = new char[size];
   memcpy(buffer_copy, buffer, size);
-  scoped_ptr<char> scoped_buffer_copy(buffer_copy);
-  return new BinaryValue(scoped_buffer_copy.Pass(), size);
+  return new BinaryValue(buffer_copy, size);
 }
 
 BinaryValue* BinaryValue::DeepCopy() const {
-  return CreateWithCopiedBuffer(buffer_.get(), size_);
+  return CreateWithCopiedBuffer(buffer_, size_);
 }
 
 bool BinaryValue::Equals(const Value* other) const {
@@ -332,9 +333,16 @@
   const BinaryValue* other_binary = static_cast<const BinaryValue*>(other);
   if (other_binary->size_ != size_)
     return false;
-  return !memcmp(GetBuffer(), other_binary->GetBuffer(), size_);
+  return !memcmp(buffer_, other_binary->buffer_, size_);
 }
 
+BinaryValue::BinaryValue(char* buffer, size_t size)
+  : Value(TYPE_BINARY),
+    buffer_(buffer),
+    size_(size) {
+  DCHECK(buffer_);
+}
+
 ///////////////////// DictionaryValue ////////////////////
 
 DictionaryValue::DictionaryValue()