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

Unified Diff: chrome/browser/browser.cc

Issue 518085: Mac: temporarily disable Full Screen and Task Manager. (Closed)
Patch Set: bug numbers added to TODOs Created 10 years, 11 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browser.cc
diff --git a/chrome/browser/browser.cc b/chrome/browser/browser.cc
index ce931ccceda30678ef36baf2901793fafa999212..91f01015726e5cf55c8e63b8cca3c90af97c1412 100644
--- a/chrome/browser/browser.cc
+++ b/chrome/browser/browser.cc
@@ -2480,7 +2480,11 @@ void Browser::InitCommandState() {
command_updater_.UpdateCommandEnabled(IDC_NEW_TAB, true);
command_updater_.UpdateCommandEnabled(IDC_CLOSE_TAB, true);
command_updater_.UpdateCommandEnabled(IDC_DUPLICATE_TAB, true);
+// TODO(viettrungluu): Temporarily disabled on Mac. Must disable here (not in
+// BWC) so that it also affects the wrench menu. http://crbug.com/31638
+#if !defined(OS_MACOSX)
command_updater_.UpdateCommandEnabled(IDC_FULLSCREEN, true);
Nico 2010/01/08 23:08:04 shouldn't this be "false"? also below. (If not, t
viettrungluu 2010/01/08 23:09:58 It's an #ifndef. Commands are disabled by default.
Nico 2010/01/08 23:11:22 doh. Carry on.
+#endif
command_updater_.UpdateCommandEnabled(IDC_EXIT, true);
#if defined(TOOLKIT_VIEWS)
command_updater_.UpdateCommandEnabled(IDC_COMPACT_NAVBAR, true);
@@ -2551,7 +2555,11 @@ void Browser::InitCommandState() {
command_updater_.UpdateCommandEnabled(IDC_CREATE_SHORTCUTS, false);
command_updater_.UpdateCommandEnabled(IDC_DEV_TOOLS, true);
command_updater_.UpdateCommandEnabled(IDC_DEV_TOOLS_CONSOLE, true);
+// TODO(viettrungluu): Temporarily disabled on Mac. Must disable here (not in
+// BWC) so that it also affects the page menu. http://crbug.com/13156
+#if !defined(OS_MACOSX)
command_updater_.UpdateCommandEnabled(IDC_TASK_MANAGER, true);
+#endif
command_updater_.UpdateCommandEnabled(IDC_SELECT_PROFILE, true);
command_updater_.UpdateCommandEnabled(IDC_SHOW_HISTORY, true);
command_updater_.UpdateCommandEnabled(IDC_SHOW_BOOKMARK_MANAGER, true);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698