Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(346)

Unified Diff: chrome/renderer/extensions/webstore_bindings.h

Issue 2791533002: Convert Web Store Inline Install IPCs to mojo (Closed)
Patch Set: Addressed Devlin's comments Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/common/extensions/typemaps.gni ('k') | chrome/renderer/extensions/webstore_bindings.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..09c5dacbcfc997b17fbf7745e33fa15da4198981 100644
--- a/chrome/renderer/extensions/webstore_bindings.h
+++ b/chrome/renderer/extensions/webstore_bindings.h
@@ -7,39 +7,39 @@
#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"
+#include "v8/include/v8.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 mojom::InlineInstallProgressListener {
public:
explicit WebstoreBindings(ScriptContext* context);
+ ~WebstoreBindings() override;
- // IPC::Listener
- bool OnMessageReceived(const IPC::Message& message) override;
+ // mojom::InlineInstallProgressListener:
+ void InlineInstallStageChanged(api::webstore::InstallStage stage) override;
+ void InlineInstallDownloadProgress(int percent_downloaded) override;
private:
+ void InlineInstallResponse(int install_id,
+ bool success,
+ const std::string& error,
+ webstore_install::Result result);
void Install(const v8::FunctionCallbackInfo<v8::Value>& args);
- void OnInlineWebstoreInstallResponse(int install_id,
- bool success,
- const std::string& error,
- webstore_install::Result result);
-
- void OnInlineInstallStageChanged(int stage);
-
- void OnInlineInstallDownloadProgress(int percent_downloaded);
-
// 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
// frame. On success, true will be returned and the |webstore_item_id|
@@ -51,6 +51,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);
};
« no previous file with comments | « chrome/common/extensions/typemaps.gni ('k') | chrome/renderer/extensions/webstore_bindings.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698