Add "New Window" and "New Incognito Window" items to the Dock Menu....

chrome/browser/app_controller_mac.mm

 // Copyright (c) 2009 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.
 
 #import "chrome/browser/app_controller_mac.h"
 
-#include "app/l10n_util.h"
+#include "app/l10n_util_mac.h"
 #include "base/command_line.h"
 #include "base/mac_util.h"
 #include "base/message_loop.h"
 #include "base/sys_string_conversions.h"
 #include "chrome/app/chrome_dll_resource.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_init.h"
 #include "chrome/browser/browser_list.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/browser_shutdown.h"
@@ skipping 23 matching lines @@
 - (void)initMenuState;
 - (void)openURLs:(const std::vector<GURL>&)urls;
 - (void)openPendingURLs;
 - (void)getUrl:(NSAppleEventDescriptor*)event
      withReply:(NSAppleEventDescriptor*)reply;
 - (void)openFiles:(NSAppleEventDescriptor*)event
         withReply:(NSAppleEventDescriptor*)reply;
 - (void)windowLayeringDidChange:(NSNotification*)inNotification;
 - (BOOL)userWillWaitForInProgressDownloads:(int)downloadCount;
 - (BOOL)shouldQuitWithInProgressDownloads;
+- (NSMenu*)applicationDockMenu:(NSApplication*)sender;
 @end
 
 @implementation AppController
 
 // This method is called very early in application startup (ie, before
 // the profile is loaded or any preferences have been registered). Defer any
 // user-data initialization until -applicationDidFinishLaunching:.
 - (void)awakeFromNib {
   pendingURLs_.reset(new std::vector<GURL>());
 
@@ skipping 598 matching lines @@
         addObserver:self
            selector:@selector(aboutWindowClosed:)
                name:kUserClosedAboutNotification
              object:aboutController_.get()];
   }
   if (![[aboutController_ window] isVisible])
     [[aboutController_ window] center];
   [aboutController_ showWindow:self];
 }
 
+- (NSMenu*)applicationDockMenu:(id)sender {
+  NSMenu* result = [[[NSMenu alloc] initWithTitle: @""] autorelease];

[pink (ping after 24hrs), 2009/09/14 19:13:03]

rather than autoreleasing, we prefer to use scoped_nsobject<T> since we have it
in Chromium.

[John Grabowski, 2009/09/14 21:22:22]

For this line here, I don't think so.
If it was a scoped_nsobject the returned value would be invalid before being
used by the caller.
Given the necessary use of autorelease, it makes some sense to follow the
pattern of autorelease instead of scoped_nsobject in the rest of the function.

Yes, we could scope_nsobject it, then retain+autorelease before returning it,
but it isn't clear that is better.

[sgk, 2009/09/14 22:10:53]

In fact, it does fail as you'd expect (message sent to freed object) if you just
replace this line with a scoped_nsobject initialization.

Following up with a CL for the other stylistic changes.

+  NSString* titleStr;

[pink (ping after 24hrs), 2009/09/14 19:13:03]

these should be declared on the same line as they are initialized.

+  id item;
+
+  titleStr = l10n_util::GetNSStringWithFixup(IDS_NEW_WINDOW_MAC);
+  item = [[[NSMenuItem alloc] initWithTitle:titleStr
+                                     action:@selector(commandDispatch:)
+                              keyEquivalent:@""] autorelease];
+  [item setTarget:self];
+  [item setTag:IDC_NEW_WINDOW];
+  [result addItem:item];
+
+  titleStr = l10n_util::GetNSStringWithFixup(IDS_NEW_INCOGNITO_WINDOW_MAC);
+  item = [[[NSMenuItem alloc] initWithTitle:titleStr
+                                     action:@selector(commandDispatch:)
+                              keyEquivalent:@""] autorelease];
+  [item setTarget:self];
+  [item setTag:IDC_NEW_INCOGNITO_WINDOW];
+  [result addItem:item];
+
+  return result;
+}
+
 @end