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/iframe_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/iframe_uitest.cc')
-rw-r--r-- | chrome/browser/iframe_uitest.cc | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/chrome/browser/iframe_uitest.cc b/chrome/browser/iframe_uitest.cc index 15a1b11..d255344 100644 --- a/chrome/browser/iframe_uitest.cc +++ b/chrome/browser/iframe_uitest.cc @@ -4,7 +4,6 @@ #include "base/basictypes.h" #include "base/file_path.h" -#include "base/platform_thread.h" #include "build/build_config.h" #include "chrome/test/ui/ui_test.h" #include "net/base/net_util.h" @@ -16,13 +15,7 @@ class IFrameTest : public UITest { test_file = test_file.AppendASCII(url); NavigateToURL(net::FilePathToFileURL(test_file)); - // The browser lazily updates the title. - PlatformThread::Sleep(sleep_timeout_ms()); - - // Make sure the navigation succeeded. EXPECT_EQ(std::wstring(page_title), GetActiveTabTitle()); - - // UITest will check if this crashed. } }; |