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

extensions/browser/api/storage/storage_api.cc

Index: extensions/browser/api/storage/storage_api.cc
diff --git a/extensions/browser/api/storage/storage_api.cc b/extensions/browser/api/storage/storage_api.cc
index 4b1928e471112682d329028b1099c4a257f98106..8b526f86c366de0df076bfcfcb73c48b4df71c27 100644
--- a/extensions/browser/api/storage/storage_api.cc
+++ b/extensions/browser/api/storage/storage_api.cc
@@ -78,9 +78,9 @@
   if (!result->status().ok())
     return Error(result->status().message);
 
-  base::DictionaryValue* dict = new base::DictionaryValue();
+  std::unique_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
   dict->Swap(&result->settings());
-  return OneArgument(dict);
+  return OneArgument(std::move(dict));
 }
 
 ExtensionFunction::ResponseValue SettingsFunction::UseWriteResult(
@@ -217,7 +217,7 @@
   }
 
   return OneArgument(
-      new base::FundamentalValue(static_cast<int>(bytes_in_use)));
+      base::MakeUnique<base::FundamentalValue>(static_cast<int>(bytes_in_use)));
 }
 
 ExtensionFunction::ResponseValue StorageStorageAreaSetFunction::RunWithStorage(