diff options
| author | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-03-30 15:22:45 +0000 |
|---|---|---|
| committer | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-03-30 15:22:45 +0000 |
| commit | 076bf500c06a2ec9ce9f5accc0eaad7c0b0b2666 (patch) | |
| tree | 90c75e38f905641cda5eff21273c76e9f80b4ca5 /chrome/test/automated_ui_tests | |
| parent | 09247ce046ae7fa18a868d7f07ad292899b3798c (diff) | |
| download | chromium_src-076bf500c06a2ec9ce9f5accc0eaad7c0b0b2666.zip chromium_src-076bf500c06a2ec9ce9f5accc0eaad7c0b0b2666.tar.gz chromium_src-076bf500c06a2ec9ce9f5accc0eaad7c0b0b2666.tar.bz2 | |
Cleanup in automation. Remove WaitForTabCountToChange.
WaitForTabCountToBecome is easier to use and more reliable.
Also removed Windows2000-specific code from accessibility unit tests.
Review URL: http://codereview.chromium.org/45070
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12774 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/test/automated_ui_tests')
| -rw-r--r-- | chrome/test/automated_ui_tests/automated_ui_tests.cc | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/chrome/test/automated_ui_tests/automated_ui_tests.cc b/chrome/test/automated_ui_tests/automated_ui_tests.cc index 0567d26..287e38e 100644 --- a/chrome/test/automated_ui_tests/automated_ui_tests.cc +++ b/chrome/test/automated_ui_tests/automated_ui_tests.cc @@ -455,11 +455,9 @@ bool AutomatedUITest::CloseActiveTab() { automation()->GetBrowserWindowCount(&browser_windows_count); // Avoid quitting the application by not closing the last window. if (tab_count > 1) { - int new_tab_count; return_value = browser->RunCommand(IDC_CLOSE_TAB); // Wait for the tab to close before we continue. - if (!browser->WaitForTabCountToChange(tab_count, - &new_tab_count, + if (!browser->WaitForTabCountToBecome(tab_count - 1, action_max_timeout_ms())) { AddWarningAttribute("tab_count_failed_to_change"); return false; |