summaryrefslogtreecommitdiffstats
path: root/chrome/browser/extensions/execute_script_apitest.cc
diff options
context:
space:
mode:
authorphajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-02-08 14:31:53 +0000
committerphajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-02-08 14:31:53 +0000
commit1b45ddfc45c9715c8f15a22e8d39d05c355bf379 (patch)
treedf791dd992974e9e4ab20112035db3bbca79a21f /chrome/browser/extensions/execute_script_apitest.cc
parentef3c87b1e44a5f67a6568e830ae31baf62166ab7 (diff)
downloadchromium_src-1b45ddfc45c9715c8f15a22e8d39d05c355bf379.zip
chromium_src-1b45ddfc45c9715c8f15a22e8d39d05c355bf379.tar.gz
chromium_src-1b45ddfc45c9715c8f15a22e8d39d05c355bf379.tar.bz2
Mark ExtensionApiTest.ExecuteScript DISABLED because it's not only flaky,
but crashy and it has been ignored for weeks! It's EXTREMELY flaky, #1 flaky test of all time with over 600 flips in the last three weeks. This test seriously needs some love. TBR=rafaelw TEST=none BUG=28630 Review URL: http://codereview.chromium.org/574030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38355 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/execute_script_apitest.cc')
-rw-r--r--chrome/browser/extensions/execute_script_apitest.cc6
1 files changed, 3 insertions, 3 deletions
diff --git a/chrome/browser/extensions/execute_script_apitest.cc b/chrome/browser/extensions/execute_script_apitest.cc
index 4c7aa87..4ef3d81 100644
--- a/chrome/browser/extensions/execute_script_apitest.cc
+++ b/chrome/browser/extensions/execute_script_apitest.cc
@@ -5,9 +5,9 @@
#include "chrome/browser/extensions/extension_apitest.h"
#include "net/base/mock_host_resolver.h"
-// This test failed at times on the Vista dbg builder and has been marked as
-// flaky for now. Bug http://code.google.com/p/chromium/issues/detail?id=28630
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, FLAKY_ExecuteScript) {
+// EXTREMELY flaky, crashy, and bad. See http://crbug.com/28630 and don't dare
+// to re-enable without a real fix or at least adding more debugging info.
+IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_ExecuteScript) {
// We need a.com to be a little bit slow to trigger a race condition.
host_resolver()->AddRuleWithLatency("a.com", "127.0.0.1", 500);
host_resolver()->AddRule("b.com", "127.0.0.1");