Add remaining items to Chrome App menus. (Mac)

chrome/browser/ui/cocoa/apps/app_shim_menu_controller_mac.mm

Index: chrome/browser/ui/cocoa/apps/app_shim_menu_controller_mac.mm
diff --git a/chrome/browser/ui/cocoa/apps/app_shim_menu_controller_mac.mm b/chrome/browser/ui/cocoa/apps/app_shim_menu_controller_mac.mm
index 52afc07c6afa54d09981b52b4055d0c32f4c3225..04ae3ea98e477c5f3d01e9e42cf48b4e9f59a509 100644
--- a/chrome/browser/ui/cocoa/apps/app_shim_menu_controller_mac.mm
+++ b/chrome/browser/ui/cocoa/apps/app_shim_menu_controller_mac.mm
@@ -170,6 +170,14 @@ void AddDuplicateItem(NSMenuItem* top_level_item,
 }
 
 - (void)buildAppMenuItems {
+  aboutDoppelganger_.reset([[DoppelgangerMenuItem alloc]
+      initWithController:self
+                 menuTag:IDC_CHROME_MENU
+                 itemTag:IDC_ABOUT
+              resourceId:IDS_ABOUT_MAC
+                  action:nil
+           keyEquivalent:@""]);
+  [[aboutDoppelganger_ menuItem] setEnabled:NO];  // Not implemented yet.
   hideDoppelganger_.reset([[DoppelgangerMenuItem alloc]
       initWithController:self
                  menuTag:IDC_CHROME_MENU
@@ -184,6 +192,21 @@ void AddDuplicateItem(NSMenuItem* top_level_item,
               resourceId:IDS_EXIT_MAC
                   action:@selector(quitCurrentPlatformApp)
            keyEquivalent:@"q"]);
+  newDoppelganger_.reset([[DoppelgangerMenuItem alloc]

[tapted, 2013/09/18 07:11:45]

should this be setEnabled:NO as well? Same for Open.

[jackhou1, 2013/09/19 03:05:05]

Not needed because only appMenuItem_ has autoenablesItems turned off. Moved the
setEnabled line lower down where it's more obvious.

+      initWithController:self
+                 menuTag:IDC_FILE_MENU
+                 itemTag:IDC_NEW_WINDOW
+              resourceId:0
+                  action:nil
+           keyEquivalent:@"n"]);
+  [[newDoppelganger_ menuItem] setTitle:l10n_util::GetNSString(IDS_NEW_MAC)];

[tapted, 2013/09/18 07:11:45]

Maybe a comment here, saying that, for apps, the "Window" part of "New Window"
is dropped to match the default menu set given to Cocoa Apps.

[jackhou1, 2013/09/19 03:05:05]

Done.

+  openDoppelganger_.reset([[DoppelgangerMenuItem alloc]
+      initWithController:self
+                 menuTag:IDC_FILE_MENU
+                 itemTag:IDC_OPEN_FILE
+              resourceId:0
+                  action:nil
+           keyEquivalent:@"o"]);
   allToFrontDoppelganger_.reset([[DoppelgangerMenuItem alloc]
       initWithController:self
                  menuTag:IDC_WINDOW_MENU
@@ -200,16 +223,28 @@ void AddDuplicateItem(NSMenuItem* top_level_item,
   [appMenuItem_ setSubmenu:appMenu];
   [appMenu setAutoenablesItems:NO];
 
+  [appMenu addItem:[aboutDoppelganger_ menuItem]];
+  [appMenu addItem:[NSMenuItem separatorItem]];
   [appMenu addItem:[hideDoppelganger_ menuItem]];
   [appMenu addItem:[NSMenuItem separatorItem]];
   [appMenu addItem:[quitDoppelganger_ menuItem]];
 
   // File menu.
   fileMenuItem_.reset([NewTopLevelItemFrom(IDC_FILE_MENU) retain]);
+  [[fileMenuItem_ submenu] addItem:[newDoppelganger_ menuItem]];
+  [[fileMenuItem_ submenu] addItem:[openDoppelganger_ menuItem]];
+  [[fileMenuItem_ submenu] addItem:[NSMenuItem separatorItem]];
   AddDuplicateItem(fileMenuItem_, IDC_FILE_MENU, IDC_CLOSE_WINDOW);
 
-  // Edit menu. This is copied entirely.
+  // Edit menu. This copies the menu entirely and removes
+  // "Paste and Match Style" and "Find". This is because the last two items,
+  // "Start Dictation" and "Special Characters" are added by OSX, so we can copy

[tapted, 2013/09/18 07:11:45]

This comment doesn't make clear to me the connection between Start Dictation /
Special Characters, and removing the two other items.

[jackhou1, 2013/09/19 03:05:05]

Oops, it should say "can't copy them".

+  // them explicitly.
   editMenuItem_.reset([[[NSApp mainMenu] itemWithTag:IDC_EDIT_MENU] copy]);
+  NSMenu* editMenu = [editMenuItem_ submenu];
+  [editMenu removeItem:[editMenu
+      itemWithTag:IDC_CONTENT_CONTEXT_PASTE_AND_MATCH_STYLE]];
+  [editMenu removeItem:[editMenu itemWithTag:IDC_FIND_MENU]];
 
   // Window menu.
   windowMenuItem_.reset([NewTopLevelItemFrom(IDC_WINDOW_MENU) retain]);
@@ -277,8 +312,11 @@ void AddDuplicateItem(NSMenuItem* top_level_item,
   for (NSMenuItem* item in [mainMenu itemArray])
     [item setHidden:YES];
 
+  [aboutDoppelganger_ enableForApp:app];
   [hideDoppelganger_ enableForApp:app];
   [quitDoppelganger_ enableForApp:app];
+  [newDoppelganger_ enableForApp:app];
+  [openDoppelganger_ enableForApp:app];
 
   [appMenuItem_ setTitle:appId];
   [[appMenuItem_ submenu] setTitle:title];
@@ -305,8 +343,11 @@ void AddDuplicateItem(NSMenuItem* top_level_item,
   for (NSMenuItem* item in [mainMenu itemArray])
     [item setHidden:NO];
 
+  [aboutDoppelganger_ disable];
   [hideDoppelganger_ disable];
   [quitDoppelganger_ disable];
+  [newDoppelganger_ disable];
+  [openDoppelganger_ disable];
 }
 
 - (void)quitCurrentPlatformApp {