Drive extension functions from ExtensionFunction::Run. The

extensions/browser/api/storage/storage_api.h

Index: extensions/browser/api/storage/storage_api.h
diff --git a/extensions/browser/api/storage/storage_api.h b/extensions/browser/api/storage/storage_api.h
index 9f1091c6870e4c68be0e59d57dde8e0548796570..b4b97da6289bed4714507c0e1f2a5a76059d7eb1 100644
--- a/extensions/browser/api/storage/storage_api.h
+++ b/extensions/browser/api/storage/storage_api.h
@@ -22,7 +22,7 @@ class SettingsFunction : public UIThreadExtensionFunction {
 
   // ExtensionFunction:
   virtual bool ShouldSkipQuotaLimiting() const OVERRIDE;
-  virtual ResponseAction RunImplTypesafe() OVERRIDE;
+  virtual ResponseAction Run() OVERRIDE;

[not at google - send to devlin, 2014/04/30 15:25:46]

Once again the Storage API is the only one which uses the new function type.

 
   // Extension settings function implementations should do their work here.
   // The StorageFrontend makes sure this is posted to the appropriate thread.
@@ -48,7 +48,7 @@ class SettingsFunction : public UIThreadExtensionFunction {
                                ValueStore* storage);
 
  private:
-  // Called via PostTask from RunImplTypesafe. Calls RunWithStorage and then
+  // Called via PostTask from Run. Calls RunWithStorage and then
   // SendResponse with its success value.
   void AsyncRunWithStorage(ValueStore* storage);