Make element removal methods in DictionaryValue and ListValue take scoped_ptr's as outparams.

chrome/browser/value_store/value_store_frontend.cc

Index: chrome/browser/value_store/value_store_frontend.cc
diff --git a/chrome/browser/value_store/value_store_frontend.cc b/chrome/browser/value_store/value_store_frontend.cc
index e53e1d1e72573ed253e5099e20aab9e286c49169..3930d4520970f7e7d492f130f48617ef65179bbc 100644
--- a/chrome/browser/value_store/value_store_frontend.cc
+++ b/chrome/browser/value_store/value_store_frontend.cc
@@ -39,17 +39,17 @@ class ValueStoreFrontend::Backend : public base::RefCountedThreadSafe<Backend> {
 
     // Extract the value from the ReadResult and pass ownership of it to the
     // callback.
-    base::Value* value = NULL;
-    if (!result->HasError())
+    scoped_ptr<base::Value> value;
+    if (!result->HasError()) {
       result->settings()->RemoveWithoutPathExpansion(key, &value);
-    else
+    } else {
       LOG(INFO) << "Reading " << key << " from " << db_path_.value()
-          << " failed: " << result->error();
+                << " failed: " << result->error();
+    }
 
-    scoped_ptr<base::Value> passed_value(value);
     BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
         base::Bind(&ValueStoreFrontend::Backend::RunCallback,
-                   this, callback, base::Passed(&passed_value)));
+                   this, callback, base::Passed(&value)));
   }
 
   void Set(const std::string& key, scoped_ptr<base::Value> value) {