Revision: 22303
Author:   [email protected]
Date:     Wed Jul  9 12:48:08 2014 UTC
Log:      Fix number of times failures are rerun.

BUG=374134
LOG=n
[email protected]
[email protected]

Review URL: https://codereview.chromium.org/379833003
http://code.google.com/p/v8/source/detail?r=22303

Modified:
 /branches/bleeding_edge/tools/testrunner/local/execution.py

=======================================
--- /branches/bleeding_edge/tools/testrunner/local/execution.py Wed Jul 9 11:37:36 2014 UTC +++ /branches/bleeding_edge/tools/testrunner/local/execution.py Wed Jul 9 12:48:08 2014 UTC
@@ -104,9 +104,9 @@
return Job(command, dep_command, test.id, timeout, self.context.verbose)

   def _MaybeRerun(self, pool, test):
-    if test.run <= self.context.rerun_failures_count + 1:
+    if test.run <= self.context.rerun_failures_count:
       # Possibly rerun this test if its run count is below the maximum per
-      # test. +1 as the flag controls reruns not including the first run.
+      # test. <= as the flag controls reruns not including the first run.
       if test.run == 1:
         # Count the overall number of reran tests on the first rerun.
         if self.reran_tests < self.context.rerun_failures_max:

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to