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

extensions/browser/api/storage/storage_frontend.cc

 // Copyright 2014 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 "extensions/browser/api/storage/storage_frontend.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 31 matching lines @@
       : browser_context_(context) {}
 
   // SettingsObserver implementation.
   void OnSettingsChanged(const std::string& extension_id,
                          settings_namespace::Namespace settings_namespace,
                          const std::string& change_json) override {
     // TODO(gdk): This is a temporary hack while the refactoring for
     // string-based event payloads is removed. http://crbug.com/136045
     std::unique_ptr<base::ListValue> args(new base::ListValue());
     args->Append(base::JSONReader::Read(change_json));
-    args->Append(new base::StringValue(settings_namespace::ToString(
-        settings_namespace)));
+    args->AppendString(settings_namespace::ToString(settings_namespace));
     std::unique_ptr<Event> event(new Event(events::STORAGE_ON_CHANGED,
                                            api::storage::OnChanged::kEventName,
                                            std::move(args)));
     EventRouter::Get(browser_context_)
         ->DispatchEventToExtension(extension_id, std::move(event));
   }
 
  private:
   BrowserContext* const browser_context_;
 };
@@ skipping 114 matching lines @@
 // static
 BrowserContextKeyedAPIFactory<StorageFrontend>*
 StorageFrontend::GetFactoryInstance() {
   return g_factory.Pointer();
 }
 
 // static
 const char* StorageFrontend::service_name() { return "StorageFrontend"; }
 
 }  // namespace extensions