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

extensions/browser/api/storage/storage_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/storage/storage_api.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <utility>
@@ skipping 60 matching lines @@
       BrowserThread::UI,
       FROM_HERE,
       base::Bind(&SettingsFunction::Respond, this, base::Passed(&response)));
 }
 
 ExtensionFunction::ResponseValue SettingsFunction::UseReadResult(
     ValueStore::ReadResult result) {
   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(
     ValueStore::WriteResult result) {
   if (!result->status().ok())
     return Error(result->status().message);
 
   if (!result->changes().empty()) {
     observers_->Notify(FROM_HERE, &SettingsObserver::OnSettingsChanged,
                        extension_id(), settings_namespace_,
@@ skipping 116 matching lines @@
                          &as_string_list);
       bytes_in_use = storage->GetBytesInUse(as_string_list);
       break;
     }
 
     default:
       return BadMessage();
   }
 
   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(
     ValueStore* storage) {
   base::DictionaryValue* input = NULL;
   if (!args_->GetDictionary(0, &input))
     return BadMessage();
   return UseWriteResult(storage->Set(ValueStore::DEFAULTS, *input));
 }
 
@@ skipping 36 matching lines @@
 StorageStorageAreaClearFunction::RunWithStorage(ValueStore* storage) {
   return UseWriteResult(storage->Clear());
 }
 
 void StorageStorageAreaClearFunction::GetQuotaLimitHeuristics(
     QuotaLimitHeuristics* heuristics) const {
   GetModificationQuotaLimitHeuristics(heuristics);
 }
 
 }  // namespace extensions