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

Unified Diff: chrome/browser/chromeos/arc/arc_intent_helper_bridge.h

Issue 1760773004: Add "Open with <ARC-app-name>" items to the context menu (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: return std::pair Created 4 years, 9 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
Index: chrome/browser/chromeos/arc/arc_intent_helper_bridge.h
diff --git a/chrome/browser/chromeos/arc/arc_intent_helper_bridge.h b/chrome/browser/chromeos/arc/arc_intent_helper_bridge.h
index bc89532cd54e23289f2c7987dda5deb8210975b3..cbb9492c583f7ba96250191de50e0e93ea512154 100644
--- a/chrome/browser/chromeos/arc/arc_intent_helper_bridge.h
+++ b/chrome/browser/chromeos/arc/arc_intent_helper_bridge.h
@@ -4,8 +4,10 @@
#ifndef CHROME_BROWSER_CHROMEOS_ARC_ARC_INTENT_HELPER_BRIDGE_H_
#define CHROME_BROWSER_CHROMEOS_ARC_ARC_INTENT_HELPER_BRIDGE_H_
+#include "ash/renderer_context_menu/open_with_menu_controller.h"
#include "base/macros.h"
#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/chromeos/arc/open_with_menu_controller_delegate.h"
#include "chrome/browser/chromeos/arc/settings_bridge.h"
#include "components/arc/arc_bridge_service.h"
#include "components/arc/arc_service.h"
@@ -18,6 +20,7 @@ namespace arc {
class ArcIntentHelperBridge : public ArcService,
public ArcBridgeService::Observer,
public IntentHelperHost,
+ public ash::OpenWithMenuController::Delegate,
public SettingsBridge::Delegate {
public:
explicit ArcIntentHelperBridge(ArcBridgeService* bridge_service);
@@ -35,9 +38,18 @@ class ArcIntentHelperBridge : public ArcService,
// arc::IntentHelperHost
void OnOpenUrl(const mojo::String& url) override;
+ // ash::OpenWithMenuController::Delegate
+ int PopulateOpenWithMenu(RenderViewContextMenuProxy* proxy,
+ int menu_id_start,
+ size_t num_menu_items,
+ const GURL& url) override;
+ void ExecuteCommand(int id) override;
+ void MenuClosed() override;
+
private:
mojo::Binding<IntentHelperHost> binding_;
scoped_ptr<SettingsBridge> settings_bridge_;
+ scoped_ptr<OpenWithMenuControllerDelegate> menu_delegate_impl_;
DISALLOW_COPY_AND_ASSIGN(ArcIntentHelperBridge);
};

Powered by Google App Engine
This is Rietveld 408576698