Remove ListValue::Append(new {Fundamental,String}Value(...)) pattern in //extensions

extensions/browser/quota_service_unittest.cc

Index: extensions/browser/quota_service_unittest.cc
diff --git a/extensions/browser/quota_service_unittest.cc b/extensions/browser/quota_service_unittest.cc
index dd73849be113dcf7c99919901da76a34cd72f74f..70a9346e38448248b60d2f470f5e0ea711b8e3ab 100644
--- a/extensions/browser/quota_service_unittest.cc
+++ b/extensions/browser/quota_service_unittest.cc
@@ -180,14 +180,14 @@ TEST_F(QuotaServiceTest, NoHeuristic) {
 TEST_F(QuotaServiceTest, FrozenHeuristic) {
   scoped_refptr<MockFunction> f(new FrozenMockFunction("foo"));
   base::ListValue args;
-  args.Append(new base::FundamentalValue(1));
+  args.AppendInteger(1);
   EXPECT_NE("", service_->Assess(extension_a_, f.get(), &args, kStartTime));
 }
 
 TEST_F(QuotaServiceTest, SingleHeuristic) {
   scoped_refptr<MockFunction> f(new TimedLimitMockFunction("foo"));
   base::ListValue args;
-  args.Append(new base::FundamentalValue(1));
+  args.AppendInteger(1);
   EXPECT_EQ("", service_->Assess(extension_a_, f.get(), &args, kStartTime));
   EXPECT_EQ("",
             service_->Assess(extension_a_,
@@ -201,8 +201,8 @@ TEST_F(QuotaServiceTest, SingleHeuristic) {
                              kStartTime + TimeDelta::FromSeconds(15)));
 
   base::ListValue args2;
-  args2.Append(new base::FundamentalValue(1));
-  args2.Append(new base::FundamentalValue(2));
+  args2.AppendInteger(1);
+  args2.AppendInteger(2);
   EXPECT_EQ("", service_->Assess(extension_b_, f.get(), &args2, kStartTime));
   EXPECT_EQ("",
             service_->Assess(extension_b_,
@@ -226,7 +226,7 @@ TEST_F(QuotaServiceTest, SingleHeuristic) {
 
   // Test that items are independent.
   base::ListValue args3;
-  args3.Append(new base::FundamentalValue(3));
+  args3.AppendInteger(3);
   EXPECT_EQ("", service_->Assess(extension_c_, f.get(), &args, kStartTime));
   EXPECT_EQ("",
             service_->Assess(extension_c_,
@@ -261,8 +261,8 @@ TEST_F(QuotaServiceTest, MultipleFunctionsDontInterfere) {
 
   base::ListValue args_f;
   base::ListValue args_g;
-  args_f.Append(new base::FundamentalValue(1));
-  args_g.Append(new base::FundamentalValue(2));
+  args_f.AppendInteger(1);
+  args_g.AppendInteger(2);
 
   EXPECT_EQ("", service_->Assess(extension_a_, f.get(), &args_f, kStartTime));
   EXPECT_EQ("", service_->Assess(extension_a_, g.get(), &args_g, kStartTime));
@@ -291,7 +291,7 @@ TEST_F(QuotaServiceTest, MultipleFunctionsDontInterfere) {
 TEST_F(QuotaServiceTest, ViolatorsWillBeForgiven) {
   scoped_refptr<MockFunction> f(new TimedLimitMockFunction("foo"));
   base::ListValue arg;
-  arg.Append(new base::FundamentalValue(1));
+  arg.AppendInteger(1);
   EXPECT_EQ("", service_->Assess(extension_a_, f.get(), &arg, kStartTime));
   EXPECT_EQ("",
             service_->Assess(extension_a_,