diff options
| author | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-26 16:12:10 +0000 |
|---|---|---|
| committer | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-26 16:12:10 +0000 |
| commit | 9d6dcdf0882b57fbb0825ab880f321d7682bad4a (patch) | |
| tree | 57bad051c7aa8c37234256a7bf163f71741b1098 /chrome/browser/errorpage_uitest.cc | |
| parent | c9b19949487f406436c9f5cdfdf35ce3bd805d67 (diff) | |
| download | chromium_src-9d6dcdf0882b57fbb0825ab880f321d7682bad4a.zip chromium_src-9d6dcdf0882b57fbb0825ab880f321d7682bad4a.tar.gz chromium_src-9d6dcdf0882b57fbb0825ab880f321d7682bad4a.tar.bz2 | |
[GTTF] Various ui tests cleanups:
- unify timeout handling
- remove unneccessary assertions that clutter the code
They were mostly there because earlier the automation
framework was less reliable. Now the same checks (or
equivalent) are always done in the framework itself.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/1377001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@42767 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/errorpage_uitest.cc')
| -rw-r--r-- | chrome/browser/errorpage_uitest.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/browser/errorpage_uitest.cc b/chrome/browser/errorpage_uitest.cc index 6c1f34f..efd7135 100644 --- a/chrome/browser/errorpage_uitest.cc +++ b/chrome/browser/errorpage_uitest.cc @@ -12,10 +12,10 @@ class ErrorPageTest : public UITest { protected: bool WaitForTitleMatching(const std::wstring& title) { - for (int i = 0; i < 100; ++i) { + for (int i = 0; i < 10; ++i) { if (GetActiveTabTitle() == title) return true; - PlatformThread::Sleep(sleep_timeout_ms() / 10); + PlatformThread::Sleep(sleep_timeout_ms()); } EXPECT_EQ(title, GetActiveTabTitle()); return false; |