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

chrome/browser/net/predictor.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/net/predictor.h"
 
 #include <algorithm>
 #include <cmath>
 #include <set>
 #include <sstream>
@@ skipping 570 matching lines @@
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   // Just finish up work if an incremental trim is in progress.
   if (urls_being_trimmed_.empty())
     LoadUrlsForTrimming();
   IncrementalTrimReferrers(true);  // Do everything now.
 }
 
 void Predictor::SerializeReferrers(base::ListValue* referral_list) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   referral_list->Clear();
-  referral_list->Append(new base::FundamentalValue(kPredictorReferrerVersion));
+  referral_list->AppendInteger(kPredictorReferrerVersion);
   for (Referrers::const_iterator it = referrers_.begin();
        it != referrers_.end(); ++it) {
     // Serialize the list of subresource names.
     base::Value* subresource_list(it->second.Serialize());
 
     // Create a list for each referer.
     base::ListValue* motivator(new base::ListValue);
-    motivator->Append(new base::StringValue(it->first.spec()));
+    motivator->AppendString(it->first.spec());
     motivator->Append(subresource_list);
 
     referral_list->Append(motivator);
   }
 }
 
 void Predictor::DeserializeReferrers(const base::ListValue& referral_list) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   int format_version = -1;
   if (referral_list.GetSize() > 0 &&
@@ skipping 631 matching lines @@
   if (first_navigations_.find(url) == first_navigations_.end())
     first_navigations_[url] = base::TimeTicks::Now();
 }
 
 void Predictor::InitialObserver::GetInitialDnsResolutionList(
     base::ListValue* startup_list) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   DCHECK(startup_list);
   startup_list->Clear();
   DCHECK_EQ(0u, startup_list->GetSize());
-  startup_list->Append(
-      new base::FundamentalValue(kPredictorStartupFormatVersion));
+  startup_list->AppendInteger(kPredictorStartupFormatVersion);
   for (FirstNavigations::iterator it = first_navigations_.begin();
        it != first_navigations_.end();
        ++it) {
     DCHECK(it->first == Predictor::CanonicalizeUrl(it->first));
-    startup_list->Append(new base::StringValue(it->first.spec()));
+    startup_list->AppendString(it->first.spec());
   }
 }
 
 void Predictor::InitialObserver::GetFirstResolutionsHtml(
     std::string* output) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   UrlInfo::UrlInfoTable resolution_list;
   {
     for (FirstNavigations::iterator it(first_navigations_.begin());
@@ skipping 46 matching lines @@
 }
 
 void SimplePredictor::ShutdownOnUIThread() {
   SetShutdown(true);
 }
 
 bool SimplePredictor::CanPrefetchAndPrerender() const { return true; }
 bool SimplePredictor::CanPreresolveAndPreconnect() const { return true; }
 
 }  // namespace chrome_browser_net