diff options
author | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-31 21:01:33 +0000 |
---|---|---|
committer | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-31 21:01:33 +0000 |
commit | 694af78d7d4251ca079216b188d01c8ecbdf23d6 (patch) | |
tree | 26fe40adc01185926428ea5144ebb38bb0575811 /content/public/test | |
parent | c63c87a8b27ece629b4c0d2a42fe9704ad952ae4 (diff) | |
download | chromium_src-694af78d7d4251ca079216b188d01c8ecbdf23d6.zip chromium_src-694af78d7d4251ca079216b188d01c8ecbdf23d6.tar.gz chromium_src-694af78d7d4251ca079216b188d01c8ecbdf23d6.tar.bz2 |
GTTF: Make the new test launcher exit early if many tests are broken.
Also tag the JSON results indicating this was an early exit.
BUG=236893
R=sky@chromium.org
Review URL: https://codereview.chromium.org/53353005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@232208 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/public/test')
-rw-r--r-- | content/public/test/test_launcher.cc | 18 |
1 files changed, 0 insertions, 18 deletions
diff --git a/content/public/test/test_launcher.cc b/content/public/test/test_launcher.cc index 97b2343..e70c15f 100644 --- a/content/public/test/test_launcher.cc +++ b/content/public/test/test_launcher.cc @@ -90,8 +90,6 @@ class WrapperTestLauncherDelegate : public base::TestLauncherDelegate { WrapperTestLauncherDelegate(content::TestLauncherDelegate* launcher_delegate, size_t jobs) : launcher_delegate_(launcher_delegate), - timeout_count_(0), - printed_timeout_message_(false), parallel_launcher_(jobs) { CHECK(temp_dir_.CreateUniqueTempDir()); } @@ -130,13 +128,6 @@ class WrapperTestLauncherDelegate : public base::TestLauncherDelegate { content::TestLauncherDelegate* launcher_delegate_; - // Number of times a test timeout occurred. - size_t timeout_count_; - - // True after a message about too many timeouts has been printed, - // to avoid doing it more than once. - bool printed_timeout_message_; - base::ParallelTestLauncher parallel_launcher_; // Store dependent test name (map is indexed by full test name). @@ -191,15 +182,6 @@ bool WrapperTestLauncherDelegate::ShouldRunTest( return false; } - // Stop test execution after too many timeouts. - if (timeout_count_ > 5) { - if (!printed_timeout_message_) { - printed_timeout_message_ = true; - printf("Too many timeouts, aborting test\n"); - } - return false; - } - return true; } |