From 9f1d28b7547a293c1c78aa8c18e962979e8612a4 Mon Sep 17 00:00:00 2001 From: "phajdan.jr@chromium.org" Date: Mon, 9 Mar 2009 18:20:05 +0000 Subject: Make SecondLaunch UI test not-flaky by using more reliable automation calls. BUG=5671 Review URL: http://codereview.chromium.org/40301 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11259 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/app/chrome_main_uitest.cc | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/chrome/app/chrome_main_uitest.cc b/chrome/app/chrome_main_uitest.cc index 6038c9c..b70e58c 100644 --- a/chrome/app/chrome_main_uitest.cc +++ b/chrome/app/chrome_main_uitest.cc @@ -35,17 +35,13 @@ TEST_F(ChromeMainTest, AppTestingInterface) { } // Make sure that the second invocation creates a new window. -// This test is disabled. See bug 5671. -TEST_F(ChromeMainTest, DISABLED_SecondLaunch) { +TEST_F(ChromeMainTest, SecondLaunch) { include_testing_id_ = false; use_existing_browser_ = true; LaunchBrowser(CommandLine(L""), false); - int window_count; - ASSERT_TRUE(automation()->WaitForWindowCountToChange(1, &window_count, - action_timeout_ms())); - ASSERT_EQ(2, window_count); + ASSERT_TRUE(automation()->WaitForWindowCountToBecome(2, action_timeout_ms())); } TEST_F(ChromeMainTest, ReuseBrowserInstanceWhenOpeningFile) { -- cgit v1.1