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).

[markusheintz_, 2013/01/17 15:13:08]

If this is for Mac should use IF DEFS ?

[Avi (use Gerrit), 2013/01/17 18:35:11]

We have IDC_OPEN_FILE on other platforms; looks like a bad (or at least out of
date) comment to me.

[markusheintz_, 2013/01/18 12:54:07]

I see. Could you update the comment and remove the "(Mac-only)"? Thanks :)

On 2013/01/17 18:35:11, Avi wrote:

[Avi (use Gerrit), 2013/01/18 19:19:23]

That's the plan; I'm OOO now and will do so when I return.

+  static void UpdateOpenFileState(CommandUpdater* command_updater);
+
+  // Update commands whose state depends on incognito mode availability and that
+  // only depend on the profile.
+  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.