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

Unified Diff: chrome/browser/ui/toolbar/media_router_contextual_menu.h

Issue 2155293002: Show the Cast toolbar icon ephemerally when Cast is in use (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 4 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/ui/toolbar/media_router_contextual_menu.h
diff --git a/chrome/browser/ui/toolbar/media_router_contextual_menu.h b/chrome/browser/ui/toolbar/media_router_contextual_menu.h
index 234f7852597e4be46c1ef75b09ec94a17e130c61..be20e9a6e1847d11a869ac88155bc22eff01ca28 100644
--- a/chrome/browser/ui/toolbar/media_router_contextual_menu.h
+++ b/chrome/browser/ui/toolbar/media_router_contextual_menu.h
@@ -10,6 +10,7 @@
#include "ui/base/models/simple_menu_model.h"
class Browser;
+class MediaRouterAction;
// The class for the contextual menu for the Media Router action.
class MediaRouterContextualMenu : public ui::SimpleMenuModel::Delegate {
@@ -17,6 +18,8 @@ class MediaRouterContextualMenu : public ui::SimpleMenuModel::Delegate {
explicit MediaRouterContextualMenu(Browser* browser);
~MediaRouterContextualMenu() override;
+ void SetMediaRouterAction(MediaRouterAction* action);
Devlin 2016/08/02 20:18:51 Is there a reason to not pass this in in the ctor?
takumif 2016/08/03 22:03:51 No. Putting it in the constructor.
+
ui::MenuModel* menu_model() { return &menu_model_; }
private:
@@ -27,10 +30,10 @@ class MediaRouterContextualMenu : public ui::SimpleMenuModel::Delegate {
void ExecuteCommand(int command_id, int event_flags) override;
void ReportIssue();
- void RemoveMediaRouterComponentAction();
Browser* browser_;
ui::SimpleMenuModel menu_model_;
+ MediaRouterAction* action_;
DISALLOW_COPY_AND_ASSIGN(MediaRouterContextualMenu);
};

Powered by Google App Engine
This is Rietveld 408576698