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

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

Issue 2559323002: Add a policy to always show the Cast toolbar icon (Closed)
Patch Set: Created 4 years 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 32bd1cac45ab204017e035887707173d9bc0896b..aa57490abe2dcdc816ddee335088185cb1898849 100644
--- a/chrome/browser/ui/toolbar/media_router_contextual_menu.h
+++ b/chrome/browser/ui/toolbar/media_router_contextual_menu.h
@@ -19,6 +19,8 @@ class ComponentMigrationHelper;
class MediaRouterContextualMenu : public ui::SimpleMenuModel::Delegate {
public:
explicit MediaRouterContextualMenu(Browser* browser);
+ // Constructor for injecting values in tests.
imcheng 2016/12/15 19:57:38 nit: empty line above
takumif 2016/12/15 22:04:14 Done.
+ MediaRouterContextualMenu(Browser* browser, bool shown_by_policy);
~MediaRouterContextualMenu() override;
ui::MenuModel* menu_model() { return &menu_model_; }
@@ -28,6 +30,8 @@ class MediaRouterContextualMenu : public ui::SimpleMenuModel::Delegate {
ToggleCloudServicesItem);
FRIEND_TEST_ALL_PREFIXES(MediaRouterContextualMenuUnitTest,
ToggleAlwaysShowIconItem);
+ FRIEND_TEST_ALL_PREFIXES(MediaRouterContextualMenuUnitTest,
+ ActionShownByPolicy);
// Gets or sets the "Always show icon" option.
bool GetAlwaysShowActionPref() const;
@@ -45,6 +49,9 @@ class MediaRouterContextualMenu : public ui::SimpleMenuModel::Delegate {
ui::SimpleMenuModel menu_model_;
extensions::ComponentMigrationHelper* const component_migration_helper_;
+ // Whether the media router action is shown by an administrator policy.
+ bool shown_by_policy_;
+
DISALLOW_COPY_AND_ASSIGN(MediaRouterContextualMenu);
};

Powered by Google App Engine
This is Rietveld 408576698