Reland of ExtensionFunction: don't pass ownership of base::Value by raw pointer.

chrome/browser/extensions/api/instance_id/instance_id_api.cc

Index: chrome/browser/extensions/api/instance_id/instance_id_api.cc
diff --git a/chrome/browser/extensions/api/instance_id/instance_id_api.cc b/chrome/browser/extensions/api/instance_id/instance_id_api.cc
index 57609b3a8538cf1a9bb2f911a4d5194ada36956d..3baab243e3dd3acbfe5ba5e85a1ed0f80281a963 100644
--- a/chrome/browser/extensions/api/instance_id/instance_id_api.cc
+++ b/chrome/browser/extensions/api/instance_id/instance_id_api.cc
@@ -5,6 +5,7 @@
 #include "chrome/browser/extensions/api/instance_id/instance_id_api.h"
 
 #include "base/logging.h"
+#include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/services/gcm/instance_id/instance_id_profile_service.h"
@@ -94,7 +95,7 @@
 }
 
 void InstanceIDGetIDFunction::GetIDCompleted(const std::string& id) {
-  Respond(OneArgument(new base::StringValue(id)));
+  Respond(OneArgument(base::MakeUnique<base::StringValue>(id)));
 }
 
 InstanceIDGetCreationTimeFunction::InstanceIDGetCreationTimeFunction() {}
@@ -110,7 +111,8 @@
 
 void InstanceIDGetCreationTimeFunction::GetCreationTimeCompleted(
     const base::Time& creation_time) {
-  Respond(OneArgument(new base::FundamentalValue(creation_time.ToDoubleT())));
+  Respond(OneArgument(
+      base::MakeUnique<base::FundamentalValue>(creation_time.ToDoubleT())));
 }
 
 InstanceIDGetTokenFunction::InstanceIDGetTokenFunction() {}
@@ -139,7 +141,7 @@
     const std::string& token,
     instance_id::InstanceID::Result result) {
   if (result == instance_id::InstanceID::SUCCESS)
-    Respond(OneArgument(new base::StringValue(token)));
+    Respond(OneArgument(base::MakeUnique<base::StringValue>(token)));
   else
     Respond(Error(InstanceIDResultToError(result)));
 }