diff options
| author | rafaelw@chromium.org <rafaelw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-25 17:45:16 +0000 |
|---|---|---|
| committer | rafaelw@chromium.org <rafaelw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-25 17:45:16 +0000 |
| commit | c290a6237ec644fc77c9328c0756be76b8316be8 (patch) | |
| tree | 3c4dc9ff93706c806f7fdafd4dc37278aff020c6 /chrome/browser/extensions/extension_override_apitest.cc | |
| parent | 1549593d29cb99d623d2c81bb83a5f01f60f7f0f (diff) | |
| download | chromium_src-c290a6237ec644fc77c9328c0756be76b8316be8.zip chromium_src-c290a6237ec644fc77c9328c0756be76b8316be8.tar.gz chromium_src-c290a6237ec644fc77c9328c0756be76b8316be8.tar.bz2 | |
ExtensionApiTest improvements.
This fixes a race condition where ExtensionBrowserTest::WaitForExtensionHostsToLoad() could have exited before all hosts were loaded. It simplifies the Overrides test. It also adds some debug output for aiding the hunt for remaining flakiness.
BUG=22668
Review URL: http://codereview.chromium.org/220039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27213 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/extension_override_apitest.cc')
| -rw-r--r-- | chrome/browser/extensions/extension_override_apitest.cc | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/chrome/browser/extensions/extension_override_apitest.cc b/chrome/browser/extensions/extension_override_apitest.cc index 8d313f8..0ebe460 100644 --- a/chrome/browser/extensions/extension_override_apitest.cc +++ b/chrome/browser/extensions/extension_override_apitest.cc @@ -8,12 +8,6 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, Overrides) { // The first pass response is the creation of a new tab. ASSERT_TRUE(RunExtensionTest("override1")) << message_; - // The overridden new tab page also sends a pass response. - WaitForPassFail(); - - // There should be no additional pass/fail responses. - EXPECT_EQ(results_.size(), 0U); - // TODO(erikkay) load a second override and verify behavior, then unload // the first and verify behavior, etc. } |