Remove use of deprecated base::ListValue::Append(Value*) overload in extensions.

chrome/browser/extensions/api/system_private/system_private_api.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/extensions/api/system_private/system_private_api.h"
 
+#include <memory>
 #include <utility>
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/values.h"
 #include "build/build_config.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/extensions/event_router_forwarder.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/extensions/api/system_private.h"
@@ skipping 30 matching lines @@
 const char kNotAvailableState[] = "NotAvailable";
 const char kNeedRestartState[] = "NeedRestart";
 
 #if defined(OS_CHROMEOS)
 const char kUpdatingState[] = "Updating";
 #endif  // defined(OS_CHROMEOS)
 
 // Dispatches an extension event with |argument|
 void DispatchEvent(extensions::events::HistogramValue histogram_value,
                    const std::string& event_name,
-                   base::Value* argument) {
+                   std::unique_ptr<base::Value> argument) {
   std::unique_ptr<base::ListValue> list_args(new base::ListValue());
   if (argument) {
-    list_args->Append(argument);
+    list_args->Append(std::move(argument));
   }
   g_browser_process->extension_event_router_forwarder()
       ->BroadcastEventToRenderers(histogram_value, event_name,
                                   std::move(list_args), GURL());
 }
 
 }  // namespace
 
 namespace extensions {
 
@@ skipping 71 matching lines @@
   dict->SetDouble(kDownloadProgressKey, download_progress);
   return RespondNow(OneArgument(std::move(dict)));
 }
 
 ExtensionFunction::ResponseAction SystemPrivateGetApiKeyFunction::Run() {
   return RespondNow(OneArgument(
       base::MakeUnique<base::StringValue>(google_apis::GetAPIKey())));
 }
 
 void DispatchVolumeChangedEvent(double volume, bool is_volume_muted) {
-  base::DictionaryValue* dict = new base::DictionaryValue();
+  std::unique_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
   dict->SetDouble(kVolumeKey, volume);
   dict->SetBoolean(kIsVolumeMutedKey, is_volume_muted);
   DispatchEvent(extensions::events::SYSTEM_PRIVATE_ON_VOLUME_CHANGED,
-                system_private::OnVolumeChanged::kEventName, dict);
+                system_private::OnVolumeChanged::kEventName, std::move(dict));
 }
 
 void DispatchBrightnessChangedEvent(int brightness, bool user_initiated) {
-  base::DictionaryValue* dict = new base::DictionaryValue();
+  std::unique_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
   dict->SetInteger(kBrightnessKey, brightness);
   dict->SetBoolean(kUserInitiatedKey, user_initiated);
   DispatchEvent(extensions::events::SYSTEM_PRIVATE_ON_BRIGHTNESS_CHANGED,
-                system_private::OnBrightnessChanged::kEventName, dict);
+                system_private::OnBrightnessChanged::kEventName,
+                std::move(dict));
 }
 
 void DispatchScreenUnlockedEvent() {
   DispatchEvent(extensions::events::SYSTEM_PRIVATE_ON_SCREEN_UNLOCKED,
-                system_private::OnScreenUnlocked::kEventName, NULL);
+                system_private::OnScreenUnlocked::kEventName, nullptr);
 }
 
 void DispatchWokeUpEvent() {
   DispatchEvent(extensions::events::SYSTEM_PRIVATE_ON_WOKE_UP,
-                system_private::OnWokeUp::kEventName, NULL);
+                system_private::OnWokeUp::kEventName, nullptr);
 }
 
 }  // namespace extensions