Remove ExtensionFunction::SetResult(T*) overload.

extensions/browser/extension_function.h

Index: extensions/browser/extension_function.h
diff --git a/extensions/browser/extension_function.h b/extensions/browser/extension_function.h
index 0bf876410a02f18583d060e3a1d529db621fa8a5..3428ee7ddc0fdfbb96639e3535d75e27abc61bf3 100644
--- a/extensions/browser/extension_function.h
+++ b/extensions/browser/extension_function.h
@@ -201,12 +201,11 @@ class ExtensionFunction
   virtual void OnQuotaExceeded(const std::string& violation_error);
 
   // Specifies the raw arguments to the function, as a JSON value.
+  // TODO(dcheng): This should take a const ref.
   virtual void SetArgs(const base::ListValue* args);
 
   // Sets a single Value as the results of the function.
   void SetResult(std::unique_ptr<base::Value> result);
-  // As above, but deprecated. TODO(estade): remove.
-  void SetResult(base::Value* result);
 
   // Sets multiple Values as the results of the function.
   void SetResultList(std::unique_ptr<base::ListValue> results);