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

extensions/browser/extension_function.cc

Index: extensions/browser/extension_function.cc
diff --git a/extensions/browser/extension_function.cc b/extensions/browser/extension_function.cc
index 98ef88f9ca8cd02a8c68a04814ca032b73a84721..a6a4b98ea4030979897a168ecb5e804cc3c347c8 100644
--- a/extensions/browser/extension_function.cc
+++ b/extensions/browser/extension_function.cc
@@ -291,24 +291,19 @@ ExtensionFunction::ResponseValue ExtensionFunction::NoArguments() {
 }
 
 ExtensionFunction::ResponseValue ExtensionFunction::OneArgument(
-    base::Value* arg) {
+    std::unique_ptr<base::Value> arg) {
   std::unique_ptr<base::ListValue> args(new base::ListValue());
-  args->Append(arg);
+  args->Append(std::move(arg));
   return ResponseValue(new ArgumentListResponseValue(name(), "OneArgument",
                                                      this, std::move(args)));
 }
 
-ExtensionFunction::ResponseValue ExtensionFunction::OneArgument(
-    std::unique_ptr<base::Value> arg) {
-  return OneArgument(arg.release());
-}
-
 ExtensionFunction::ResponseValue ExtensionFunction::TwoArguments(
-    base::Value* arg1,
-    base::Value* arg2) {
+    std::unique_ptr<base::Value> arg1,
+    std::unique_ptr<base::Value> arg2) {
   std::unique_ptr<base::ListValue> args(new base::ListValue());
-  args->Append(arg1);
-  args->Append(arg2);
+  args->Append(std::move(arg1));
+  args->Append(std::move(arg2));
   return ResponseValue(new ArgumentListResponseValue(name(), "TwoArguments",
                                                      this, std::move(args)));
 }