Test suite that monitors the disposition of JS-created windows based on what

chrome/browser/ui/browser_browsertest.cc

 // 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.
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/file_path.h"
 #include "base/sys_info.h"
@@ skipping 168 matching lines @@
   }
 
   virtual std::string GetHTMLContents() OVERRIDE {
     return "<h1>INTERSTITIAL</h1>";
   }
 
  private:
   InterstitialPage* interstitial_page_;  // Owns us.
 };
 
+static const FilePath::CharType* kTestDir = FILE_PATH_LITERAL("click_modifier");

[sky, 2012/10/23 23:09:28]

Move this close to your tests.

[ericu, 2012/10/23 23:15:44]

Done.

+static const char kFirstPageTitle[] = "First window";
+static const char kSecondPageTitle[] = "New window!";
+
+class ClickModifierTest : public InProcessBrowserTest {
+ public:
+  ClickModifierTest() {
+  }
+
+  // Returns a url that opens a new window or tab when clicked, via javascript.
+  GURL GetTestURL() {
+    return ui_test_utils::GetTestUrl(
+      FilePath(kTestDir),
+      FilePath(FILE_PATH_LITERAL("window_open.html")));
+  }
+
+  string16 getFirstPageTitle() {
+    return ASCIIToUTF16(kFirstPageTitle);
+  }
+
+  string16 getSecondPageTitle() {
+    return ASCIIToUTF16(kSecondPageTitle);
+  }
+
+  // Loads our test page and simulates a single click using the supplied button
+  // and modifiers.  The test page will call window.open, creating either a new
+  // tab or a new window.  In the case of a tab, it will verify whether the
+  // tab's in the foreground or background.
+  void RunTest(

[sky, 2012/10/23 23:09:28]

nit: wrap at the ( and move the first arg up to this line.

[ericu, 2012/10/23 23:15:44]

Done.

+      Browser* browser,
+      int modifiers,
+      WebKit::WebMouseEvent::Button button,
+      bool expect_new_window,
+      bool expect_foreground) {
+

[sky, 2012/10/23 23:09:28]

nit: remove newline.

[ericu, 2012/10/23 23:15:44]

Done.

+    GURL url(GetTestURL());
+    ui_test_utils::NavigateToURL(browser, url);
+    EXPECT_EQ(1u, browser::GetBrowserCount(browser->profile()));
+    EXPECT_EQ(1, browser->tab_count());
+    content::WebContents* web_contents = chrome::GetActiveWebContents(browser);
+    EXPECT_EQ(url, web_contents->GetURL());
+
+    content::WindowedNotificationObserver observer(
+        chrome::NOTIFICATION_TAB_ADDED,
+        content::NotificationService::AllSources());
+    SimulateMouseClick(web_contents, modifiers, button);
+    observer.Wait();
+
+    unsigned expected_browser_count = expect_new_window ? 2 : 1;
+    EXPECT_EQ(expected_browser_count,
+        browser::GetBrowserCount(browser->profile()));
+
+    // If we didn't pop up a new window, we need to make sure the right tab's in
+    // front.
+    if (!expect_new_window) {
+      EXPECT_EQ(2, browser->tab_count());
+      web_contents = chrome::GetActiveWebContents(browser);
+      WaitForLoadStop(web_contents);
+      if (expect_foreground) {
+        EXPECT_EQ(getSecondPageTitle(), web_contents->GetTitle());
+      } else {
+        EXPECT_EQ(getFirstPageTitle(), web_contents->GetTitle());
+      }
+    }
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ClickModifierTest);
+};
+
+
 }  // namespace
 
 class BrowserTest : public ExtensionBrowserTest {
  protected:
   // In RTL locales wrap the page title with RTL embedding characters so that it
   // matches the value returned by GetWindowTitle().
   string16 LocaleWindowCaptionFromPageTitle(const string16& expected_title) {
     string16 page_title = WindowCaptionFromPageTitle(expected_title);
 #if defined(OS_WIN)
     std::string locale = g_browser_process->GetApplicationLocale();
@@ skipping 1462 matching lines @@
   EXPECT_EQ(0u, BrowserList::size());
 
   // Starting a browser window should work just fine.
   ui_test_utils::BrowserAddedObserver browser_added_observer;
   CreateBrowser(ProfileManager::GetDefaultProfile());
   browser_added_observer.WaitForSingleNewBrowser();
 
   EXPECT_EQ(1u, BrowserList::size());
 }
 
-// This test needs to be placed outside the anonymouse namespace because we
+// This test needs to be placed outside the anonymous namespace because we
 // need to access private type of Browser.
 class AppModeTest : public BrowserTest {
  public:
   AppModeTest() {}
 
   virtual void SetUpCommandLine(CommandLine* command_line) {
     GURL url = ui_test_utils::GetTestUrl(
        FilePath(), FilePath().AppendASCII("title1.html"));
     command_line->AppendSwitchASCII(switches::kApp, url.spec());
   }
@@ skipping 13 matching lines @@
   ASSERT_GT(ui_test_utils::FindInPage(tab, ASCIIToUTF16("WebKit"), true, true,
                                       NULL, NULL),
             0);
   ASSERT_GT(ui_test_utils::FindInPage(tab, ASCIIToUTF16("OS"), true, true,
                                       NULL, NULL),
             0);
   ASSERT_GT(ui_test_utils::FindInPage(tab, ASCIIToUTF16("JavaScript"), true,
                                       true, NULL, NULL),
             0);
 }
+
+IN_PROC_BROWSER_TEST_F(ClickModifierTest, BasicClickTest) {
+  int modifiers = 0;
+  WebKit::WebMouseEvent::Button button = WebKit::WebMouseEvent::ButtonLeft;
+  bool expect_new_window = false;
+  bool expect_foreground = true;
+  RunTest(browser(), modifiers, button, expect_new_window, expect_foreground);
+}
+
+// TODO(ericu): Alt-click behavior is platform-dependent and not well defined.
+// Should we add tests so we know if it changes?
+
+// On non-OSX platforms we ignore meta, so this should be just like
+// BasicClickTest.  On OSX, meta-click replaces control-click as a way to
+// send a link to a background tab.
+IN_PROC_BROWSER_TEST_F(ClickModifierTest, MetaClickTest) {
+  int modifiers = WebKit::WebInputEvent::MetaKey;
+  WebKit::WebMouseEvent::Button button = WebKit::WebMouseEvent::ButtonLeft;
+  bool expect_new_window = false;
+#if defined(OS_MACOSX)
+  bool expect_foreground = false;
+#else
+  bool expect_foreground = true;
+#endif
+  RunTest(browser(), modifiers, button, expect_new_window, expect_foreground);
+}
+
+// Shift-clicks open in a new window.
+// TODO(ericu): Disabled until https://bugs.webkit.org/show_bug.cgi?id=99202
+// lands, fixing http://crbug.com/31631.
+IN_PROC_BROWSER_TEST_F(ClickModifierTest, DISABLED_ShiftClickTest) {
+  int modifiers = WebKit::WebInputEvent::ShiftKey;
+  WebKit::WebMouseEvent::Button button = WebKit::WebMouseEvent::ButtonLeft;
+  bool expect_new_window = true;
+  bool expect_foreground = true;
+  RunTest(browser(), modifiers, button, expect_new_window, expect_foreground);
+}
+
+// Control-click doesn't cause a click on OSX.
+#if !defined(OS_MACOSX)
+// Control-clicks open in a background tab.
+IN_PROC_BROWSER_TEST_F(ClickModifierTest, ControlClickTest) {
+  int modifiers = WebKit::WebInputEvent::ControlKey;
+  WebKit::WebMouseEvent::Button button = WebKit::WebMouseEvent::ButtonLeft;
+  bool expect_new_window = false;
+  bool expect_foreground = false;
+  RunTest(browser(), modifiers, button, expect_new_window, expect_foreground);
+}
+#endif
+
+// Control-shift-clicks open in a foreground tab.
+IN_PROC_BROWSER_TEST_F(ClickModifierTest, ControlShiftClickTest) {
+  int modifiers = WebKit::WebInputEvent::ControlKey |
+      WebKit::WebInputEvent::ShiftKey;
+  WebKit::WebMouseEvent::Button button = WebKit::WebMouseEvent::ButtonLeft;
+  bool expect_new_window = false;
+  bool expect_foreground = true;
+  RunTest(browser(), modifiers, button, expect_new_window, expect_foreground);
+}
+
+// Middle-clicks open in a background tab.
+IN_PROC_BROWSER_TEST_F(ClickModifierTest, MiddleClickTest) {
+  int modifiers = 0;
+  WebKit::WebMouseEvent::Button button = WebKit::WebMouseEvent::ButtonMiddle;
+  bool expect_new_window = false;
+  bool expect_foreground = false;
+  RunTest(browser(), modifiers, button, expect_new_window, expect_foreground);
+}
+
+// Shift-middle-clicks open in a foreground tab.
+IN_PROC_BROWSER_TEST_F(ClickModifierTest, ShiftMiddleClickTest) {
+  int modifiers = WebKit::WebInputEvent::ShiftKey;
+  WebKit::WebMouseEvent::Button button = WebKit::WebMouseEvent::ButtonMiddle;
+  bool expect_new_window = false;
+  bool expect_foreground = true;
+  RunTest(browser(), modifiers, button, expect_new_window, expect_foreground);
+}