WebstorePrivate extension API cleanup, part 1

chrome/browser/extensions/api/webstore_private/webstore_private_api.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_WEBSTORE_PRIVATE_WEBSTORE_PRIVATE_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_WEBSTORE_PRIVATE_WEBSTORE_PRIVATE_API_H_
 
 #include <string>
 
 #include "chrome/browser/extensions/active_install_data.h"
 #include "chrome/browser/extensions/chrome_extension_function.h"
+#include "chrome/browser/extensions/chrome_extension_function_details.h"
 #include "chrome/browser/extensions/extension_install_prompt.h"
 #include "chrome/browser/extensions/webstore_install_helper.h"
 #include "chrome/browser/extensions/webstore_installer.h"
 #include "chrome/common/extensions/api/webstore_private.h"
 #include "chrome/common/extensions/webstore_install_result.h"
 #include "content/public/browser/gpu_data_manager_observer.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "google_apis/gaia/google_service_auth_error.h"
 #include "third_party/skia/include/core/SkBitmap.h"
@@ skipping 15 matching lines @@
       WebstoreInstaller::Delegate* delegate);
 
   // Gets the pending approval for the |extension_id| in |profile|. Pending
   // approvals are held between the calls to beginInstallWithManifest and
   // completeInstall. This should only be used for testing.
   static scoped_ptr<WebstoreInstaller::Approval> PopApprovalForTesting(
       Profile* profile, const std::string& extension_id);
 };
 
 class WebstorePrivateBeginInstallWithManifest3Function
-    : public ChromeAsyncExtensionFunction,
+    : public UIThreadExtensionFunction,
       public ExtensionInstallPrompt::Delegate,
       public WebstoreInstallHelper::Delegate {
  public:
   DECLARE_EXTENSION_FUNCTION("webstorePrivate.beginInstallWithManifest3",
                              WEBSTOREPRIVATE_BEGININSTALLWITHMANIFEST3)
 
   // Result codes for the return value. If you change this, make sure to
   // update the description for the beginInstallWithManifest3 callback in
   // the extension API JSON.
   enum ResultCode {
@@ skipping 19 matching lines @@
 
     // Invalid icon url.
     INVALID_ICON_URL,
 
     // An extension with the same extension id has already been installed.
     ALREADY_INSTALLED,
   };
 
   WebstorePrivateBeginInstallWithManifest3Function();
 
+ private:
+  ~WebstorePrivateBeginInstallWithManifest3Function() override;
+
+  // ExtensionFunction:
+  ExtensionFunction::ResponseAction Run() override;
+
   // WebstoreInstallHelper::Delegate:
   void OnWebstoreParseSuccess(const std::string& id,
                               const SkBitmap& icon,
                               base::DictionaryValue* parsed_manifest) override;
   void OnWebstoreParseFailure(const std::string& id,
                               InstallHelperResultCode result_code,
                               const std::string& error_message) override;
 
   // ExtensionInstallPrompt::Delegate:
   void InstallUIProceed() override;
   void InstallUIAbort(bool user_initiated) override;
 
- protected:
-  ~WebstorePrivateBeginInstallWithManifest3Function() override;
+  // Response helpers.
+  const char* ResultCodeToString(ResultCode code) const;
+  ExtensionFunction::ResponseValue BuildResponseForSuccess();
+  ExtensionFunction::ResponseValue BuildResponseForError(
+      ResultCode code, const std::string& error);
 
-  // ExtensionFunction:
-  bool RunAsync() override;
+  // Convenience response methods.
+  ExtensionFunction::ResponseAction RespondNowWithError(
+      ResultCode code, const std::string& error);
+  void RespondWithSuccess();
+  void RespondWithError(ResultCode code, const std::string& error);
 
-  // Sets the result_ as a string based on |code|.
-  void SetResultCode(ResultCode code);
+  ChromeExtensionFunctionDetails chrome_details_;
 
- private:
-  const char* ResultCodeToString(ResultCode code);
-
-  // These store the input parameters to the function.
+  // This stores the input parameters to the function.
   scoped_ptr<api::webstore_private::BeginInstallWithManifest3::Params> params_;
 
   // The results of parsing manifest_ and icon_data_ go into these two.
   scoped_ptr<base::DictionaryValue> parsed_manifest_;
   SkBitmap icon_;
 
   // A dummy Extension object we create for the purposes of using
   // ExtensionInstallPrompt to prompt for confirmation of the install.
   scoped_refptr<extensions::Extension> dummy_extension_;
 
@@ skipping 178 matching lines @@
  protected:
   ~WebstorePrivateGetEphemeralAppsEnabledFunction() override;
 
   // ExtensionFunction:
   bool RunSync() override;
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_WEBSTORE_PRIVATE_WEBSTORE_PRIVATE_API_H_