Chromium Code Reviews| 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); |
| }; |