Convert Web Store Inline Install IPCs to mojo

chrome/renderer/extensions/webstore_bindings.h

Index: chrome/renderer/extensions/webstore_bindings.h
diff --git a/chrome/renderer/extensions/webstore_bindings.h b/chrome/renderer/extensions/webstore_bindings.h
index 641fddd50888c3bbfd34d38102484fdbb2814111..7c20fc539e2bec40cd73aeb71ef9bc6916a0d0cb 100644
--- a/chrome/renderer/extensions/webstore_bindings.h
+++ b/chrome/renderer/extensions/webstore_bindings.h
@@ -7,38 +7,43 @@
 
 #include "base/compiler_specific.h"
 #include "base/macros.h"
+#include "chrome/common/extensions/mojom/inline_install.mojom.h"
 #include "chrome/common/extensions/webstore_install_result.h"
 #include "chrome/renderer/extensions/chrome_v8_extension_handler.h"
 #include "extensions/renderer/object_backed_native_handler.h"
+#include "mojo/public/cpp/bindings/binding_set.h"
 
 namespace blink {
 class WebLocalFrame;
 }
 
 namespace extensions {
+class ScriptContext;
 
 // A V8 extension that creates an object at window.chrome.webstore. This object
 // allows JavaScript to initiate inline installs of apps that are listed in the
 // Chrome Web Store (CWS).
 class WebstoreBindings : public ObjectBackedNativeHandler,
-                         public ChromeV8ExtensionHandler {
+                         public ChromeV8ExtensionHandler,
+                         public mojom::InlineInstallProgressListener {
  public:
   explicit WebstoreBindings(ScriptContext* context);
+  ~WebstoreBindings() override;
 
-  // IPC::Listener
+  // IPC::Listener:
   bool OnMessageReceived(const IPC::Message& message) override;
 
- private:
-  void Install(const v8::FunctionCallbackInfo<v8::Value>& args);
+  void InlineInstallResponse(int install_id,

[Devlin, 2017/05/24 21:33:59]

nit: This can be private

[catmullings, 2017/05/25 00:14:45]

Done.

+                             bool success,
+                             const std::string& error,
+                             webstore_install::Result result);
 
-  void OnInlineWebstoreInstallResponse(int install_id,
-                                       bool success,
-                                       const std::string& error,
-                                       webstore_install::Result result);
+  // mojom::InlineInstallProgressListener:
+  void InlineInstallStageChanged(api::webstore::InstallStage stage) override;
+  void InlineInstallDownloadProgress(int percent_downloaded) override;
 
-  void OnInlineInstallStageChanged(int stage);
-
-  void OnInlineInstallDownloadProgress(int percent_downloaded);
+ private:
+  void Install(const v8::FunctionCallbackInfo<v8::Value>& args);
 
   // Extracts a Web Store item ID from a <link rel="chrome-webstore-item"
   // href="https://chrome.google.com/webstore/detail/id"> node found in the
@@ -51,6 +56,11 @@ class WebstoreBindings : public ObjectBackedNativeHandler,
       std::string* webstore_item_id,
       std::string* error);
 
+  mojom::InlineInstallerAssociatedPtr inline_installer_;
+
+  mojo::BindingSet<mojom::InlineInstallProgressListener>
+      install_progress_listener_bindings_;
+
   DISALLOW_COPY_AND_ASSIGN(WebstoreBindings);
 };