From 0fad1cda301b8cb6f1c0f280df1d1deb381dfdb3 Mon Sep 17 00:00:00 2001 From: "csharp@chromium.org" Date: Wed, 20 Jun 2012 21:04:04 +0000 Subject: Ensure Timed out Tests Aren't Ignored BUG=128978 TEST= Review URL: https://chromiumcodereview.appspot.com/10559069 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143260 0039d316-1c4b-4281-b951-d872f2087c98 --- content/test/test_launcher.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'content/test') diff --git a/content/test/test_launcher.cc b/content/test/test_launcher.cc index a59220b..de0e1b0 100644 --- a/content/test/test_launcher.cc +++ b/content/test/test_launcher.cc @@ -497,9 +497,9 @@ bool RunTests(TestLauncherDelegate* launcher_delegate, bool ignore_failure = false; - // -1 exit code means a crash or hang. Never ignore those failures, - // they are serious and should always be visible. - if (exit_code != -1) + // Never ignore crashes or hangs/timeouts, they are serious and should + // always be visible. + if (exit_code != -1 && !was_timeout) ignore_failure = base::TestSuite::ShouldIgnoreFailure(*test_info); printer.OnTestEnd(test_info->name(), test_case->name(), true, true, -- cgit v1.1