Make Mac menu code obey incognito availability.

chrome/browser/ui/browser_command_controller.h

Index: chrome/browser/ui/browser_command_controller.h
diff --git a/chrome/browser/ui/browser_command_controller.h b/chrome/browser/ui/browser_command_controller.h
index 31bfe7bf47ad17a1d714a3d971bb85043ff6ae1c..467f6cb56225524bf25faa59b2b09bc804fa90fc 100644
--- a/chrome/browser/ui/browser_command_controller.h
+++ b/chrome/browser/ui/browser_command_controller.h
@@ -66,6 +66,18 @@ class BrowserCommandController : public CommandUpdaterDelegate,
   void PrintingStateChanged();
   void LoadingStateChanged(bool is_loading, bool force);
 
+  // Shared state updating: these functions are static and public to share with
+  // outside code.
+
+  // Updates the open-file state (Mac-only).
+  static void UpdateOpenFileState(CommandUpdater* command_updater);
+
+  // Update commands whose state depends on incognito mode availability and that
+  // only depend on the profile.

[Robert Sesek, 2013/01/16 18:24:50]

nit: dependS?

[Avi (use Gerrit), 2013/01/16 19:04:37]

The commands depend on the profile. Awkward constructions ftw.

+  static void UpdateSharedCommandsForIncognitoAvailability(
+      CommandUpdater* command_updater,
+      Profile* profile);
+
  private:
   enum FullScreenMode {
     // Not in fullscreen mode.
@@ -163,9 +175,6 @@ class BrowserCommandController : public CommandUpdaterDelegate,
   // Updates the save-page-as command state.
   void UpdateSaveAsState();
 
-  // Updates the open-file state (Mac Only).
-  void UpdateOpenFileState();
-
   // Ask the Reload/Stop button to change its icon, and update the Stop command
   // state.  |is_loading| is true if the current WebContents is loading.
   // |force| is true if the button should change its icon immediately.