Remove ExtensionFunction::SetResult(T*) overload.

chrome/browser/extensions/api/sync_file_system/sync_file_system_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/sync_file_system/sync_file_system_api.h"
 
+#include <memory>
 #include <string>
 #include <utility>
 
 #include "base/bind.h"
 #include "base/logging.h"
+#include "base/memory/ptr_util.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/extensions/api/sync_file_system/extension_sync_event_observer.h"
 #include "chrome/browser/extensions/api/sync_file_system/sync_file_system_api_helpers.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sync_file_system/sync_file_status.h"
 #include "chrome/browser/sync_file_system/sync_file_system_service.h"
 #include "chrome/browser/sync_file_system/sync_file_system_service_factory.h"
 #include "chrome/common/extensions/api/sync_file_system.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/render_frame_host.h"
@@ skipping 76 matching lines @@
         BrowserThread::UI,
         FROM_HERE,
         Bind(&SyncFileSystemDeleteFileSystemFunction::DidDeleteFileSystem, this,
              error));
     return;
   }
 
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   if (error != base::File::FILE_OK) {
     error_ = ErrorToString(sync_file_system::FileErrorToSyncStatusCode(error));
-    SetResult(new base::FundamentalValue(false));
+    SetResult(base::MakeUnique<base::FundamentalValue>(false));
     SendResponse(false);
     return;
   }
 
-  SetResult(new base::FundamentalValue(true));
+  SetResult(base::MakeUnique<base::FundamentalValue>(true));
   SendResponse(true);
 }
 
 bool SyncFileSystemRequestFileSystemFunction::RunAsync() {
   // SyncFileSystem initialization is done in OpenFileSystem below, but we call
   // GetSyncFileSystemService here too to initialize sync event observer for
   // extensions API.
   if (!GetSyncFileSystemService(GetProfile()))
     return false;
 
@@ skipping 32 matching lines @@
     return;
   }
 
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   if (error != base::File::FILE_OK) {
     error_ = ErrorToString(sync_file_system::FileErrorToSyncStatusCode(error));
     SendResponse(false);
     return;
   }
 
-  base::DictionaryValue* dict = new base::DictionaryValue();
-  SetResult(dict);
+  std::unique_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
   dict->SetString("name", file_system_name);
   dict->SetString("root", root_url.spec());
+  SetResult(std::move(dict));
   SendResponse(true);
 }
 
 bool SyncFileSystemGetFileStatusFunction::RunAsync() {
   std::string url;
   EXTENSION_FUNCTION_VALIDATE(args_->GetString(0, &url));
 
   scoped_refptr<storage::FileSystemContext> file_system_context =
       BrowserContext::GetStoragePartition(
           GetProfile(), render_frame_host()->GetSiteInstance())
@@ skipping 83 matching lines @@
   // Keep mapping file statuses until all of them have been received.
   // TODO(calvinlo): Get rid of this check when batch version of
   // GetFileSyncStatus(GURL urls[]); is added.
   if (num_results_received_ < num_expected_results_)
     return;
 
   // All results received. Dump array of statuses into extension enum values.
   // Note that the enum types need to be set as strings manually as the
   // autogenerated Results::Create function thinks the enum values should be
   // returned as int values.
-  base::ListValue* status_array = new base::ListValue();
+  std::unique_ptr<base::ListValue> status_array(new base::ListValue());
   for (URLToStatusMap::iterator it = file_sync_statuses_.begin();
        it != file_sync_statuses_.end(); ++it) {
     base::DictionaryValue* dict = new base::DictionaryValue();
     status_array->Append(dict);
 
     storage::FileSystemURL url = it->first;
     SyncStatusCode file_error = it->second.first;
     api::sync_file_system::FileStatus file_status =
         SyncFileStatusToExtensionEnum(it->second.second);
 
     dict->Set("entry", CreateDictionaryValueForFileSystemEntry(
         url, sync_file_system::SYNC_FILE_TYPE_FILE));
     dict->SetString("status", ToString(file_status));
 
     if (file_error == sync_file_system::SYNC_STATUS_OK)
       continue;
     dict->SetString("error", ErrorToString(file_error));
   }
-  SetResult(status_array);
+  SetResult(std::move(status_array));
 
   SendResponse(true);
 }
 
 bool SyncFileSystemGetUsageAndQuotaFunction::RunAsync() {
   std::string url;
   EXTENSION_FUNCTION_VALIDATE(args_->GetString(0, &url));
 
   scoped_refptr<storage::FileSystemContext> file_system_context =
       BrowserContext::GetStoragePartition(
@@ skipping 56 matching lines @@
       api::sync_file_system::ParseConflictResolutionPolicy(policy_string));
   if (policy != sync_file_system::CONFLICT_RESOLUTION_POLICY_LAST_WRITE_WIN) {
     SetError(base::StringPrintf(kUnsupportedConflictResolutionPolicy,
                                 policy_string.c_str()));
     return false;
   }
   return true;
 }
 
 bool SyncFileSystemGetConflictResolutionPolicyFunction::RunSync() {
-  SetResult(new base::StringValue(
-      api::sync_file_system::ToString(
-          api::sync_file_system::CONFLICT_RESOLUTION_POLICY_LAST_WRITE_WIN)));
+  SetResult(base::MakeUnique<base::StringValue>(api::sync_file_system::ToString(
+      api::sync_file_system::CONFLICT_RESOLUTION_POLICY_LAST_WRITE_WIN)));
   return true;
 }
 
 bool SyncFileSystemGetServiceStatusFunction::RunSync() {
   sync_file_system::SyncFileSystemService* service =
       GetSyncFileSystemService(GetProfile());
   if (!service)
     return false;
   results_ = api::sync_file_system::GetServiceStatus::Results::Create(
       SyncServiceStateToExtensionEnum(service->GetSyncServiceState()));
   return true;
 }
 
 }  // namespace extensions