Remove ExtensionFunction::SetResult(T*) overload.

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 fd7bae96ceb8a557067d7079a91341d16c8b4a91..9203006c5181b520beeed140429b093e29ebf28e 100644
--- a/extensions/browser/extension_function.cc
+++ b/extensions/browser/extension_function.cc
@@ -256,12 +256,7 @@ void ExtensionFunction::OnQuotaExceeded(const std::string& violation_error) {
 
 void ExtensionFunction::SetArgs(const base::ListValue* args) {
   DCHECK(!args_.get());  // Should only be called once.
-  args_.reset(args->DeepCopy());
-}
-
-void ExtensionFunction::SetResult(base::Value* result) {
-  results_.reset(new base::ListValue());
-  results_->Append(result);
+  args_ = args->CreateDeepCopy();

[dcheng, 2016/05/19 03:40:25]

A bit unrelated, but I changed it since I was in the vicinity.

 }
 
 void ExtensionFunction::SetResult(std::unique_ptr<base::Value> result) {