Convert JavascriptAlertActivatesTab to browser_tests framework.

chrome/browser/browser_uitest.cc

 // Copyright (c) 2006-2008 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 "base/file_path.h"
 #include "base/gfx/native_widget_types.h"
 #include "base/string_util.h"
 #include "base/sys_info.h"
 #include "base/values.h"
 #include "chrome/app/chrome_dll_resource.h"
@@ skipping 95 matching lines @@
 
   // Make sure session restore says we didn't crash.
   scoped_ptr<DictionaryValue> profile_prefs(GetDefaultProfilePreferences());
   ASSERT_TRUE(profile_prefs.get());
   ASSERT_TRUE(profile_prefs->GetBoolean(prefs::kSessionExitedCleanly,
                                         &exited_cleanly));
   ASSERT_TRUE(exited_cleanly);
 }
 #endif
 
-// This test is flakey, see bug 5668 for details.
-TEST_F(BrowserTest, DISABLED_JavascriptAlertActivatesTab) {
-  scoped_refptr<BrowserProxy> window(automation()->GetBrowserWindow(0));
-  int start_index;
-  ASSERT_TRUE(window->GetActiveTabIndex(&start_index));
-  ASSERT_TRUE(window->AppendTab(GURL("about:blank")));
-  int javascript_tab_index;
-  ASSERT_TRUE(window->GetActiveTabIndex(&javascript_tab_index));
-  scoped_refptr<TabProxy> javascript_tab = window->GetActiveTab();
-  ASSERT_TRUE(javascript_tab.get());
-  // Switch back to the starting tab, then send the second tab a javascript
-  // alert, which should force it to become active.
-  ASSERT_TRUE(window->ActivateTab(start_index));
-  ASSERT_TRUE(
-      javascript_tab->NavigateToURLAsync(GURL("javascript:alert('Alert!')")));
-  ASSERT_TRUE(window->WaitForTabToBecomeActive(javascript_tab_index,
-                                               action_max_timeout_ms()));
-}
-
 // Test that scripts can fork a new renderer process for a tab in a particular
 // case (which matches following a link in Gmail).  The script must open a new
 // tab, set its window.opener to null, and redirect it to a cross-site URL.
 // (Bug 1115708)
 // This test can only run if V8 is in use, and not KJS, because KJS will not
 // set window.opener to null properly.
 #ifdef CHROME_V8
 TEST_F(BrowserTest, NullOpenerRedirectForksProcess) {
   // This test only works in multi-process mode
   if (in_process_renderer())
@@ skipping 133 matching lines @@
 
   // Test that window.close() works.  Since we don't have a way of executing a
   // JS function on the page through TabProxy, reload it and use an unload
   // handler that closes the page.
   ASSERT_EQ(tab->Reload(), AUTOMATION_MSG_NAVIGATION_SUCCESS);
   ASSERT_TRUE(automation()->WaitForWindowCountToBecome(1, action_timeout_ms()));
 }
 #endif
 
 }  // namespace