Mac: temporarily disable Full Screen and Task Manager.

chrome/browser/browser.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/browser.h"
 
 #include "app/animation.h"
 #include "app/l10n_util.h"
 #include "base/base_paths.h"
 #include "base/command_line.h"
@@ skipping 2462 matching lines @@
   command_updater_.UpdateCommandEnabled(IDC_NEW_WINDOW_PROFILE_3, true);
   command_updater_.UpdateCommandEnabled(IDC_NEW_WINDOW_PROFILE_4, true);
   command_updater_.UpdateCommandEnabled(IDC_NEW_WINDOW_PROFILE_5, true);
   command_updater_.UpdateCommandEnabled(IDC_NEW_WINDOW_PROFILE_6, true);
   command_updater_.UpdateCommandEnabled(IDC_NEW_WINDOW_PROFILE_7, true);
   command_updater_.UpdateCommandEnabled(IDC_NEW_WINDOW_PROFILE_8, true);
   command_updater_.UpdateCommandEnabled(IDC_CLOSE_WINDOW, true);
   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, the function has an odd name.)

[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);
 #endif
 
   // Page-related commands
   command_updater_.UpdateCommandEnabled(IDC_CLOSE_POPUPS, true);
   command_updater_.UpdateCommandEnabled(IDC_EMAIL_PAGE_LOCATION, true);
   command_updater_.UpdateCommandEnabled(IDC_PRINT, true);
   command_updater_.UpdateCommandEnabled(IDC_ENCODING_AUTO_DETECT, true);
@@ skipping 50 matching lines @@
   command_updater_.UpdateCommandEnabled(IDC_ZOOM_MENU, true);
   command_updater_.UpdateCommandEnabled(IDC_ZOOM_PLUS, true);
   command_updater_.UpdateCommandEnabled(IDC_ZOOM_NORMAL, true);
   command_updater_.UpdateCommandEnabled(IDC_ZOOM_MINUS, true);
 
   // Show various bits of UI
   command_updater_.UpdateCommandEnabled(IDC_OPEN_FILE, true);
   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);
   command_updater_.UpdateCommandEnabled(IDC_SHOW_EXTENSION_SHELF, true);
   command_updater_.UpdateCommandEnabled(IDC_SHOW_DOWNLOADS, true);
   command_updater_.UpdateCommandEnabled(IDC_HELP_PAGE, true);
   command_updater_.UpdateCommandEnabled(IDC_IMPORT_SETTINGS, true);
 
   ExtensionsService* extensions_service = profile()->GetExtensionsService();
   bool enable_extensions =
@@ skipping 623 matching lines @@
 ///////////////////////////////////////////////////////////////////////////////
 // BrowserToolbarModel (private):
 
 NavigationController* Browser::BrowserToolbarModel::GetNavigationController() {
   // This |current_tab| can be NULL during the initialization of the
   // toolbar during window creation (i.e. before any tabs have been added
   // to the window).
   TabContents* current_tab = browser_->GetSelectedTabContents();
   return current_tab ? &current_tab->controller() : NULL;
 }