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

base/values_unittest.cc

Index: base/values_unittest.cc
diff --git a/base/values_unittest.cc b/base/values_unittest.cc
index 8776c104cc1a401ccfa50195d296577005977673..733c485d2b1aca599dcb917edf179728bef6bbc9 100644
--- a/base/values_unittest.cc
+++ b/base/values_unittest.cc
@@ -203,7 +203,7 @@ TEST(ValuesTest, ListDeletion) {
 
 TEST(ValuesTest, ListRemoval) {
   bool deletion_flag = true;
-  Value* removed_item = NULL;
+  scoped_ptr<Value> removed_item;
 
   {
     ListValue list;
@@ -218,8 +218,7 @@ TEST(ValuesTest, ListRemoval) {
     EXPECT_EQ(0U, list.GetSize());
   }
   EXPECT_FALSE(deletion_flag);
-  delete removed_item;
-  removed_item = NULL;
+  removed_item.reset();
   EXPECT_TRUE(deletion_flag);
 
   {
@@ -275,7 +274,7 @@ TEST(ValuesTest, DictionaryDeletion) {
 TEST(ValuesTest, DictionaryRemoval) {
   std::string key = "test";
   bool deletion_flag = true;
-  Value* removed_item = NULL;
+  scoped_ptr<Value> removed_item;
 
   {
     DictionaryValue dict;
@@ -288,8 +287,7 @@ TEST(ValuesTest, DictionaryRemoval) {
     ASSERT_TRUE(removed_item);
   }
   EXPECT_FALSE(deletion_flag);
-  delete removed_item;
-  removed_item = NULL;
+  removed_item.reset();
   EXPECT_TRUE(deletion_flag);
 
   {