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

Side by Side Diff: chrome/browser/ui/toolbar/media_router_contextual_menu.cc

Issue 2332693003: Show media router toolbar icon ephemerally for active local routes and issues (Closed)
Patch Set: Address Derek's comment Created 4 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <string>
6
5 #include "base/logging.h" 7 #include "base/logging.h"
6 #include "base/metrics/user_metrics.h" 8 #include "base/metrics/user_metrics.h"
7 #include "chrome/app/chrome_command_ids.h" 9 #include "chrome/app/chrome_command_ids.h"
8 #include "chrome/browser/extensions/component_migration_helper.h" 10 #include "chrome/browser/extensions/component_migration_helper.h"
9 #include "chrome/browser/media/router/media_router_factory.h" 11 #include "chrome/browser/media/router/media_router_factory.h"
10 #include "chrome/browser/media/router/mojo/media_router_mojo_impl.h" 12 #include "chrome/browser/media/router/mojo/media_router_mojo_impl.h"
11 #include "chrome/browser/profiles/profile.h" 13 #include "chrome/browser/profiles/profile.h"
12 #include "chrome/browser/ui/browser.h" 14 #include "chrome/browser/ui/browser.h"
13 #include "chrome/browser/ui/singleton_tabs.h" 15 #include "chrome/browser/ui/singleton_tabs.h"
14 #include "chrome/browser/ui/toolbar/component_toolbar_actions_factory.h" 16 #include "chrome/browser/ui/toolbar/component_toolbar_actions_factory.h"
15 #include "chrome/browser/ui/toolbar/media_router_contextual_menu.h" 17 #include "chrome/browser/ui/toolbar/media_router_contextual_menu.h"
16 #include "chrome/browser/ui/toolbar/toolbar_actions_model.h" 18 #include "chrome/browser/ui/toolbar/toolbar_actions_model.h"
19 #include "chrome/common/pref_names.h"
17 #include "chrome/common/url_constants.h" 20 #include "chrome/common/url_constants.h"
18 #include "chrome/grit/chromium_strings.h" 21 #include "chrome/grit/chromium_strings.h"
19 #include "chrome/grit/generated_resources.h" 22 #include "chrome/grit/generated_resources.h"
20 #include "extensions/common/constants.h" 23 #include "extensions/common/constants.h"
21 #include "ui/base/models/menu_model_delegate.h" 24 #include "ui/base/models/menu_model_delegate.h"
22 25
23 #if defined(GOOGLE_CHROME_BUILD) 26 #if defined(GOOGLE_CHROME_BUILD)
24 #include "chrome/browser/signin/signin_manager_factory.h" 27 #include "chrome/browser/signin/signin_manager_factory.h"
25 #include "chrome/common/pref_names.h"
26 #include "components/prefs/pref_service.h"
27 #include "components/signin/core/browser/signin_manager.h" 28 #include "components/signin/core/browser/signin_manager.h"
28 #endif // defined(GOOGLE_CHROME_BUILD) 29 #endif // defined(GOOGLE_CHROME_BUILD)
29 30
30 MediaRouterContextualMenu::MediaRouterContextualMenu(Browser* browser) 31 MediaRouterContextualMenu::MediaRouterContextualMenu(Browser* browser)
31 : browser_(browser), 32 : browser_(browser),
32 menu_model_(this) { 33 menu_model_(this),
34 component_migration_helper_(ToolbarActionsModel::Get(browser->profile())
35 ->component_migration_helper()) {
36 DCHECK(component_migration_helper_);
37
33 menu_model_.AddItemWithStringId(IDC_MEDIA_ROUTER_ABOUT, 38 menu_model_.AddItemWithStringId(IDC_MEDIA_ROUTER_ABOUT,
34 IDS_MEDIA_ROUTER_ABOUT); 39 IDS_MEDIA_ROUTER_ABOUT);
35 menu_model_.AddSeparator(ui::NORMAL_SEPARATOR); 40 menu_model_.AddSeparator(ui::NORMAL_SEPARATOR);
36 menu_model_.AddItemWithStringId(IDC_MEDIA_ROUTER_LEARN_MORE, 41 menu_model_.AddItemWithStringId(IDC_MEDIA_ROUTER_LEARN_MORE,
37 IDS_MEDIA_ROUTER_LEARN_MORE); 42 IDS_MEDIA_ROUTER_LEARN_MORE);
38 menu_model_.AddItemWithStringId(IDC_MEDIA_ROUTER_HELP, 43 menu_model_.AddItemWithStringId(IDC_MEDIA_ROUTER_HELP,
39 IDS_MEDIA_ROUTER_HELP); 44 IDS_MEDIA_ROUTER_HELP);
40 menu_model_.AddItemWithStringId(IDC_MEDIA_ROUTER_REMOVE_TOOLBAR_ACTION, 45 menu_model_.AddCheckItemWithStringId(
41 IDS_MEDIA_ROUTER_REMOVE_TOOLBAR_ACTION); 46 IDC_MEDIA_ROUTER_ALWAYS_SHOW_TOOLBAR_ACTION,
47 IDS_MEDIA_ROUTER_ALWAYS_SHOW_TOOLBAR_ACTION);
42 menu_model_.AddSeparator(ui::NORMAL_SEPARATOR); 48 menu_model_.AddSeparator(ui::NORMAL_SEPARATOR);
43 #if defined(OS_WIN) || defined(OS_MACOSX) || defined(OS_CHROMEOS) 49 #if defined(OS_WIN) || defined(OS_MACOSX) || defined(OS_CHROMEOS)
44 menu_model_.AddItemWithStringId(IDC_MEDIA_ROUTER_MANAGE_DEVICES, 50 menu_model_.AddItemWithStringId(IDC_MEDIA_ROUTER_MANAGE_DEVICES,
45 IDS_MEDIA_ROUTER_MANAGE_DEVICES); 51 IDS_MEDIA_ROUTER_MANAGE_DEVICES);
46 #endif 52 #endif
47 #if defined(GOOGLE_CHROME_BUILD) 53 #if defined(GOOGLE_CHROME_BUILD)
48 menu_model_.AddCheckItemWithStringId(IDC_MEDIA_ROUTER_CLOUD_SERVICES_TOGGLE, 54 menu_model_.AddCheckItemWithStringId(IDC_MEDIA_ROUTER_CLOUD_SERVICES_TOGGLE,
49 IDS_MEDIA_ROUTER_CLOUD_SERVICES_TOGGLE); 55 IDS_MEDIA_ROUTER_CLOUD_SERVICES_TOGGLE);
50 #endif // defined(GOOGLE_CHROME_BUILD) 56 #endif // defined(GOOGLE_CHROME_BUILD)
51 menu_model_.AddItemWithStringId(IDC_MEDIA_ROUTER_REPORT_ISSUE, 57 menu_model_.AddItemWithStringId(IDC_MEDIA_ROUTER_REPORT_ISSUE,
52 IDS_MEDIA_ROUTER_REPORT_ISSUE); 58 IDS_MEDIA_ROUTER_REPORT_ISSUE);
53 } 59 }
54 60
55 MediaRouterContextualMenu::~MediaRouterContextualMenu() { 61 MediaRouterContextualMenu::~MediaRouterContextualMenu() {}
62
63 bool MediaRouterContextualMenu::GetAlwaysShowActionPref() const {
64 return component_migration_helper_->GetComponentActionPref(
65 ComponentToolbarActionsFactory::kMediaRouterActionId);
66 }
67
68 void MediaRouterContextualMenu::SetAlwaysShowActionPref(bool always_show) {
69 component_migration_helper_->SetComponentActionPref(
70 ComponentToolbarActionsFactory::kMediaRouterActionId, always_show);
56 } 71 }
57 72
58 bool MediaRouterContextualMenu::IsCommandIdChecked(int command_id) const { 73 bool MediaRouterContextualMenu::IsCommandIdChecked(int command_id) const {
59 #if defined(GOOGLE_CHROME_BUILD) 74 #if defined(GOOGLE_CHROME_BUILD)
60 if (command_id == IDC_MEDIA_ROUTER_CLOUD_SERVICES_TOGGLE) { 75 if (command_id == IDC_MEDIA_ROUTER_CLOUD_SERVICES_TOGGLE) {
61 return browser_->profile()->GetPrefs()->GetBoolean( 76 return browser_->profile()->GetPrefs()->GetBoolean(
62 prefs::kMediaRouterEnableCloudServices); 77 prefs::kMediaRouterEnableCloudServices);
63 } 78 }
64 #endif // defined(GOOGLE_CHROME_BUILD) 79 #endif // defined(GOOGLE_CHROME_BUILD)
80 if (command_id == IDC_MEDIA_ROUTER_ALWAYS_SHOW_TOOLBAR_ACTION) {
81 return GetAlwaysShowActionPref();
82 }
65 return false; 83 return false;
66 } 84 }
67 85
68 bool MediaRouterContextualMenu::IsCommandIdEnabled(int command_id) const { 86 bool MediaRouterContextualMenu::IsCommandIdEnabled(int command_id) const {
69 return true; 87 return true;
70 } 88 }
71 89
72 bool MediaRouterContextualMenu::IsCommandIdVisible(int command_id) const { 90 bool MediaRouterContextualMenu::IsCommandIdVisible(int command_id) const {
73 #if defined(GOOGLE_CHROME_BUILD) 91 #if defined(GOOGLE_CHROME_BUILD)
74 if (command_id == IDC_MEDIA_ROUTER_CLOUD_SERVICES_TOGGLE) { 92 if (command_id == IDC_MEDIA_ROUTER_CLOUD_SERVICES_TOGGLE) {
(...skipping 16 matching lines...) Expand all
91 const char kCastLearnMorePageUrl[] = 109 const char kCastLearnMorePageUrl[] =
92 "https://support.google.com/chromecast/answer/2998338"; 110 "https://support.google.com/chromecast/answer/2998338";
93 111
94 #if defined(GOOGLE_CHROME_BUILD) 112 #if defined(GOOGLE_CHROME_BUILD)
95 PrefService* pref_service; 113 PrefService* pref_service;
96 #endif // defined(GOOGLE_CHROME_BUILD) 114 #endif // defined(GOOGLE_CHROME_BUILD)
97 switch (command_id) { 115 switch (command_id) {
98 case IDC_MEDIA_ROUTER_ABOUT: 116 case IDC_MEDIA_ROUTER_ABOUT:
99 chrome::ShowSingletonTab(browser_, GURL(kAboutPageUrl)); 117 chrome::ShowSingletonTab(browser_, GURL(kAboutPageUrl));
100 break; 118 break;
119 case IDC_MEDIA_ROUTER_ALWAYS_SHOW_TOOLBAR_ACTION:
120 SetAlwaysShowActionPref(!GetAlwaysShowActionPref());
121 break;
101 #if defined(GOOGLE_CHROME_BUILD) 122 #if defined(GOOGLE_CHROME_BUILD)
102 case IDC_MEDIA_ROUTER_CLOUD_SERVICES_TOGGLE: 123 case IDC_MEDIA_ROUTER_CLOUD_SERVICES_TOGGLE:
103 pref_service = browser_->profile()->GetPrefs(); 124 pref_service = browser_->profile()->GetPrefs();
104 pref_service->SetBoolean(prefs::kMediaRouterEnableCloudServices, 125 pref_service->SetBoolean(prefs::kMediaRouterEnableCloudServices,
105 !pref_service->GetBoolean(prefs::kMediaRouterEnableCloudServices)); 126 !pref_service->GetBoolean(prefs::kMediaRouterEnableCloudServices));
106 127
107 // If this has been set before, this is a no-op. 128 // If this has been set before, this is a no-op.
108 pref_service->SetBoolean(prefs::kMediaRouterCloudServicesPrefSet, 129 pref_service->SetBoolean(prefs::kMediaRouterCloudServicesPrefSet, true);
109 true);
110 break; 130 break;
111 #endif // defined(GOOGLE_CHROME_BUILD) 131 #endif // defined(GOOGLE_CHROME_BUILD)
112 case IDC_MEDIA_ROUTER_HELP: 132 case IDC_MEDIA_ROUTER_HELP:
113 chrome::ShowSingletonTab(browser_, GURL(kCastHelpCenterPageUrl)); 133 chrome::ShowSingletonTab(browser_, GURL(kCastHelpCenterPageUrl));
114 base::RecordAction(base::UserMetricsAction( 134 base::RecordAction(base::UserMetricsAction(
115 "MediaRouter_Ui_Navigate_Help")); 135 "MediaRouter_Ui_Navigate_Help"));
116 break; 136 break;
117 case IDC_MEDIA_ROUTER_LEARN_MORE: 137 case IDC_MEDIA_ROUTER_LEARN_MORE:
118 chrome::ShowSingletonTab(browser_, GURL(kCastLearnMorePageUrl)); 138 chrome::ShowSingletonTab(browser_, GURL(kCastLearnMorePageUrl));
119 break; 139 break;
120 #if defined(OS_WIN) || defined(OS_MACOSX) || defined(OS_CHROMEOS) 140 #if defined(OS_WIN) || defined(OS_MACOSX) || defined(OS_CHROMEOS)
121 case IDC_MEDIA_ROUTER_MANAGE_DEVICES: 141 case IDC_MEDIA_ROUTER_MANAGE_DEVICES:
122 chrome::ShowSingletonTab(browser_, GURL(chrome::kChromeUICastURL)); 142 chrome::ShowSingletonTab(browser_, GURL(chrome::kChromeUICastURL));
123 break; 143 break;
124 #endif 144 #endif
125 case IDC_MEDIA_ROUTER_REMOVE_TOOLBAR_ACTION:
126 RemoveMediaRouterComponentAction();
127 break;
128 case IDC_MEDIA_ROUTER_REPORT_ISSUE: 145 case IDC_MEDIA_ROUTER_REPORT_ISSUE:
129 ReportIssue(); 146 ReportIssue();
130 break; 147 break;
131 default: 148 default:
132 NOTREACHED(); 149 NOTREACHED();
133 } 150 }
134 } 151 }
135 152
136 void MediaRouterContextualMenu::ReportIssue() { 153 void MediaRouterContextualMenu::ReportIssue() {
137 // Opens feedback page loaded from the media router extension. 154 // Opens feedback page loaded from the media router extension.
138 // This is temporary until feedback UI is redesigned. 155 // This is temporary until feedback UI is redesigned.
139 // TODO(crbug.com/597778): remove reference to MediaRouterMojoImpl 156 // TODO(crbug.com/597778): remove reference to MediaRouterMojoImpl
140 media_router::MediaRouterMojoImpl* media_router = 157 media_router::MediaRouterMojoImpl* media_router =
141 static_cast<media_router::MediaRouterMojoImpl*>( 158 static_cast<media_router::MediaRouterMojoImpl*>(
142 media_router::MediaRouterFactory::GetApiForBrowserContext( 159 media_router::MediaRouterFactory::GetApiForBrowserContext(
143 static_cast<content::BrowserContext*>(browser_->profile()))); 160 static_cast<content::BrowserContext*>(browser_->profile())));
144 if (media_router->media_route_provider_extension_id().empty()) 161 if (media_router->media_route_provider_extension_id().empty())
145 return; 162 return;
146 std::string feedback_url(extensions::kExtensionScheme + 163 std::string feedback_url(extensions::kExtensionScheme +
147 std::string(url::kStandardSchemeSeparator) + 164 std::string(url::kStandardSchemeSeparator) +
148 media_router->media_route_provider_extension_id() + 165 media_router->media_route_provider_extension_id() +
149 "/feedback.html"); 166 "/feedback.html");
150 chrome::ShowSingletonTab(browser_, GURL(feedback_url)); 167 chrome::ShowSingletonTab(browser_, GURL(feedback_url));
151 } 168 }
152
153 void MediaRouterContextualMenu::RemoveMediaRouterComponentAction() {
154 ToolbarActionsModel::Get(browser_->profile())->component_migration_helper()
155 ->OnActionRemoved(ComponentToolbarActionsFactory::kMediaRouterActionId);
156 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698