Remove ExtensionFunction::SetResult(T*) overload.

extensions/browser/api/networking_private/networking_private_api.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/networking_private/networking_private_api.h"
 
+#include <utility>
+
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "components/onc/onc_constants.h"
 #include "extensions/browser/api/networking_private/networking_private_delegate.h"
 #include "extensions/browser/api/networking_private/networking_private_delegate_factory.h"
 #include "extensions/browser/extension_function_registry.h"
 #include "extensions/common/api/networking_private.h"
 
 namespace {
@@ skipping 40 matching lines @@
   GetDelegate(browser_context())
       ->GetProperties(
           params->network_guid,
           base::Bind(&NetworkingPrivateGetPropertiesFunction::Success, this),
           base::Bind(&NetworkingPrivateGetPropertiesFunction::Failure, this));
   return true;
 }
 
 void NetworkingPrivateGetPropertiesFunction::Success(
     std::unique_ptr<base::DictionaryValue> result) {
-  SetResult(result.release());
+  SetResult(std::move(result));
   SendResponse(true);
 }
 
 void NetworkingPrivateGetPropertiesFunction::Failure(const std::string& error) {
   error_ = error;
   SendResponse(false);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NetworkingPrivateGetManagedPropertiesFunction
@@ skipping 12 matching lines @@
           params->network_guid,
           base::Bind(&NetworkingPrivateGetManagedPropertiesFunction::Success,
                      this),
           base::Bind(&NetworkingPrivateGetManagedPropertiesFunction::Failure,
                      this));
   return true;
 }
 
 void NetworkingPrivateGetManagedPropertiesFunction::Success(
     std::unique_ptr<base::DictionaryValue> result) {
-  SetResult(result.release());
+  SetResult(std::move(result));
   SendResponse(true);
 }
 
 void NetworkingPrivateGetManagedPropertiesFunction::Failure(
     const std::string& error) {
   error_ = error;
   SendResponse(false);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NetworkingPrivateGetStateFunction
 
 NetworkingPrivateGetStateFunction::~NetworkingPrivateGetStateFunction() {
 }
 
 bool NetworkingPrivateGetStateFunction::RunAsync() {
   std::unique_ptr<private_api::GetState::Params> params =
       private_api::GetState::Params::Create(*args_);
   EXTENSION_FUNCTION_VALIDATE(params);
 
   GetDelegate(browser_context())
       ->GetState(params->network_guid,
                  base::Bind(&NetworkingPrivateGetStateFunction::Success, this),
                  base::Bind(&NetworkingPrivateGetStateFunction::Failure, this));
   return true;
 }
 
 void NetworkingPrivateGetStateFunction::Success(
     std::unique_ptr<base::DictionaryValue> result) {
-  SetResult(result.release());
+  SetResult(std::move(result));
   SendResponse(true);
 }
 
 void NetworkingPrivateGetStateFunction::Failure(const std::string& error) {
   error_ = error;
   SendResponse(false);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NetworkingPrivateSetPropertiesFunction
@@ skipping 111 matching lines @@
   GetDelegate(browser_context())
       ->GetNetworks(
           network_type, configured_only, visible_only, limit,
           base::Bind(&NetworkingPrivateGetNetworksFunction::Success, this),
           base::Bind(&NetworkingPrivateGetNetworksFunction::Failure, this));
   return true;
 }
 
 void NetworkingPrivateGetNetworksFunction::Success(
     std::unique_ptr<base::ListValue> network_list) {
-  SetResult(network_list.release());
+  SetResult(std::move(network_list));
   SendResponse(true);
 }
 
 void NetworkingPrivateGetNetworksFunction::Failure(const std::string& error) {
   error_ = error;
   SendResponse(false);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NetworkingPrivateGetVisibleNetworksFunction
@@ skipping 16 matching lines @@
           network_type, configured_only, visible_only, kDefaultNetworkListLimit,
           base::Bind(&NetworkingPrivateGetVisibleNetworksFunction::Success,
                      this),
           base::Bind(&NetworkingPrivateGetVisibleNetworksFunction::Failure,
                      this));
   return true;
 }
 
 void NetworkingPrivateGetVisibleNetworksFunction::Success(
     std::unique_ptr<base::ListValue> network_properties_list) {
-  SetResult(network_properties_list.release());
+  SetResult(std::move(network_properties_list));
   SendResponse(true);
 }
 
 void NetworkingPrivateGetVisibleNetworksFunction::Failure(
     const std::string& error) {
   error_ = error;
   SendResponse(false);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
@@ skipping 25 matching lines @@
     } else if (type == ::onc::network_type::kWimax) {
       enabled_networks_list->AppendString(
           private_api::ToString(private_api::NETWORK_TYPE_WIMAX));
     } else if (type == ::onc::network_type::kCellular) {
       enabled_networks_list->AppendString(
           private_api::ToString(private_api::NETWORK_TYPE_CELLULAR));
     } else {
       LOG(ERROR) << "networkingPrivate: Unexpected type: " << type;
     }
   }
-  SetResult(enabled_networks_list.release());
+  SetResult(std::move(enabled_networks_list));
   return true;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NetworkingPrivateGetDeviceStatesFunction
 
 NetworkingPrivateGetDeviceStatesFunction::
     ~NetworkingPrivateGetDeviceStatesFunction() {
 }
 
 bool NetworkingPrivateGetDeviceStatesFunction::RunSync() {
   std::unique_ptr<NetworkingPrivateDelegate::DeviceStateList> device_states(
       GetDelegate(browser_context())->GetDeviceStateList());
   if (!device_states) {
     error_ = networking_private::kErrorNotSupported;
     return false;
   }
 
   std::unique_ptr<base::ListValue> device_state_list(new base::ListValue);
   for (const auto& properties : *device_states)
     device_state_list->Append(properties->ToValue().release());
-  SetResult(device_state_list.release());
+  SetResult(std::move(device_state_list));
   return true;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NetworkingPrivateEnableNetworkTypeFunction
 
 NetworkingPrivateEnableNetworkTypeFunction::
     ~NetworkingPrivateEnableNetworkTypeFunction() {
 }
 
@@ skipping 386 matching lines @@
   SendResponse(true);
 }
 
 void NetworkingPrivateSetCellularSimStateFunction::Failure(
     const std::string& error) {
   error_ = error;
   SendResponse(false);
 }
 
 }  // namespace extensions