Close the ntp during startup when opening a link on Mac.

chrome/browser/app_controller_mac_browsertest.mm

 // Copyright (c) 2012 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 <Carbon/Carbon.h>
 #import <Cocoa/Cocoa.h>
+#import <Foundation/NSAppleEventDescriptor.h>
+#import <Foundation/Foundation.h>
+#import <objc/message.h>
+#import <objc/runtime.h>
 
 #include "apps/app_window_registry.h"
 #include "base/command_line.h"
 #include "base/mac/scoped_nsobject.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/app/chrome_command_ids.h"
 #import "chrome/browser/app_controller_mac.h"
 #include "chrome/browser/apps/app_browsertest_util.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/extensions/extension_test_message_listener.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_window.h"
 #import "chrome/browser/ui/cocoa/profiles/user_manager_mac.h"
 #include "chrome/browser/ui/host_desktop.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
+#include "chrome/common/url_constants.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/browser/web_contents.h"
 #include "extensions/common/extension.h"
+#include "net/test/embedded_test_server/embedded_test_server.h"
 
 namespace {
 
+GURL openShortcutUrl = GURL::EmptyGURL();

[Alexei Svitkine (slow), 2014/04/24 18:59:28]

Nit: Use c++ naming convention here.

+
+}  // namespace
+
+@interface TestOpenShortcutOnStartup : NSObject
+- (void)applicationWillFinishLaunching:(NSNotification*)notification;
+@end
+
+@implementation TestOpenShortcutOnStartup
+
+- (void)applicationWillFinishLaunching:(NSNotification*)notification {
+  if (!openShortcutUrl.is_valid())
+    return;
+
+  AppController* controller = (AppController*)[NSApp delegate];
+  Method getUrl = class_getInstanceMethod([controller class],
+      @selector(getUrl:withReply:));
+
+  if (getUrl == nil)
+    return;
+
+  base::scoped_nsobject<NSAppleEventDescriptor> shortcutEvent(
+      [[NSAppleEventDescriptor alloc]
+          initWithEventClass:kASAppleScriptSuite
+                     eventID:kASSubroutineEvent
+            targetDescriptor:nil
+                    returnID:kAutoGenerateReturnID
+               transactionID:kAnyTransactionID]);
+  [shortcutEvent setParamDescriptor:
+      [NSAppleEventDescriptor descriptorWithString:
+          [NSString stringWithUTF8String:openShortcutUrl.spec().c_str()]]
+                         forKeyword:keyDirectObject];

[Alexei Svitkine (slow), 2014/04/24 18:59:28]

Nit: Align :'s

+
+  method_invoke(controller, getUrl, shortcutEvent.get(), NULL);
+}
+
+@end
+
+namespace {
+
 class AppControllerPlatformAppBrowserTest
     : public extensions::PlatformAppBrowserTest {
  protected:
   AppControllerPlatformAppBrowserTest()
       : active_browser_list_(BrowserList::GetInstance(
                                 chrome::GetActiveDesktop())) {
   }
 
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
     PlatformAppBrowserTest::SetUpCommandLine(command_line);
@@ skipping 177 matching lines @@
   BOOL result = [ac applicationShouldHandleReopen:NSApp hasVisibleWindows:NO];
   EXPECT_FALSE(result);
 
   base::RunLoop().RunUntilIdle();
 
   EXPECT_EQ(1u, active_browser_list_->size());
   EXPECT_TRUE(UserManagerMac::IsShowing());
   UserManagerMac::Hide();
 }
 
+class AppControllerOpenShortcutBrowserTest : public InProcessBrowserTest {
+ protected:
+  AppControllerOpenShortcutBrowserTest() {
+  }
+
+  virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
+    // In order to mimic opening shortcut during browser startup, we need to
+    // send the event before applicationDidFinishLaunching is called, but

[Alexei Svitkine (slow), 2014/04/24 18:59:28]

Nit: "applicationDidFinishLaunching" -> "-applicationDidFinishLaunching". Same
below.

+    // after AppController is loaded.
+    //
+    // Since applicationWillFinishLaunching does nothing now, we swizzle it to
+    // our function to send the event. We need to do this early before running
+    // the main message loop.
+    //
+    // NSApp does not exist yet. We need to get the AppController using
+    // reflection.
+    Class appControllerClass = NSClassFromString(@"AppController");
+    Class openShortcutClass = NSClassFromString(@"TestOpenShortcutOnStartup");
+
+    // We set |constructor_success_| to false on failures.
+    if (appControllerClass == nil || openShortcutClass == nil) {
+      constructor_success_ = false;
+      return;
+    }
+
+    Method original = class_getInstanceMethod(appControllerClass,
+        @selector(applicationWillFinishLaunching:));

[Alexei Svitkine (slow), 2014/04/24 18:59:28]

Nit: Extract to local SEL variable and re-use below.

+    Method destination = class_getInstanceMethod(openShortcutClass,
+        @selector(applicationWillFinishLaunching:));
+
+    if (original == NULL || destination == NULL) {
+      constructor_success_ = false;
+      return;
+    }
+
+    method_exchangeImplementations(original, destination);
+
+    constructor_success_ =
+        embedded_test_server()->InitializeAndWaitUntilReady();
+    openShortcutUrl = embedded_test_server()->GetURL("/simple.html");
+  }
+
+  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+    // If the arg is empty, PrepareTestCommandLine() after this function will
+    // append about:blank as default url.
+    command_line->AppendArg(chrome::kChromeUINewTabURL);
+  }
+
+  bool constructor_success_ = true;

[Alexei Svitkine (slow), 2014/04/24 18:59:28]

Instead of exposing this as a protected variable, can you just add ASSERT_TRUE()
statements to SetUpInProcessBrowserTestFixture()? 

+};
+
+IN_PROC_BROWSER_TEST_F(AppControllerOpenShortcutBrowserTest,
+                       OpenShortcutOnStartup) {
+  EXPECT_TRUE(constructor_success_);
+  EXPECT_EQ(1, browser()->tab_strip_model()->count());
+  EXPECT_EQ(openShortcutUrl,
+      browser()->tab_strip_model()->GetActiveWebContents()
+          ->GetLastCommittedURL());
+}
+
 }  // namespace