[test] Simplify progress indicators.

tools/testrunner/local/execution.py

 # Copyright 2012 the V8 project authors. All rights reserved.
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 #       notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 #       copyright notice, this list of conditions and the following
 #       disclaimer in the documentation and/or other materials provided
@@ skipping 230 matching lines @@
 
       # Rerun this test.
       test.duration = None
       test.output = None
       test.run += 1
       pool.add([TestJob(test)])
       self.remaining += 1
       self.total += 1
 
   def _ProcessTestNormal(self, test, result, pool):
-    self.indicator.AboutToRun(test)
     test.output = result[1]
     test.duration = result[2]
     has_unexpected_output = test.suite.HasUnexpectedOutput(test)
     if has_unexpected_output:
       self.failed.append(test)
       if test.output.HasCrashed():
         self.crashed += 1
     else:
       self.succeeded += 1
     self.remaining -= 1
@@ skipping 16 matching lines @@
             self.printed_allocations = True
             return line
         return ""
       return (AllocationStr(output1.stdout) != AllocationStr(output2.stdout))
 
     # Always pass the test duration for the database update.
     test.duration = result[2]
     if test.run == 1 and result[1].HasTimedOut():
       # If we get a timeout in the first run, we are already in an
       # unpredictable state. Just report it as a failure and don't rerun.
-      self.indicator.AboutToRun(test)
       test.output = result[1]
       self.remaining -= 1
       self.failed.append(test)
       self.indicator.HasRun(test, True)
     if test.run > 1 and HasDifferentAllocations(test.output, result[1]):
       # From the second run on, check for different allocations. If a
       # difference is found, call the indicator twice to report both tests.
       # All runs of each test are counted as one for the statistic.
-      self.indicator.AboutToRun(test)
       self.remaining -= 1
       self.failed.append(test)
       self.indicator.HasRun(test, True)
-      self.indicator.AboutToRun(test)
       test.output = result[1]
       self.indicator.HasRun(test, True)
     elif test.run >= 3:
       # No difference on the third run -> report a success.
-      self.indicator.AboutToRun(test)
       self.remaining -= 1
       self.succeeded += 1
       test.output = result[1]
       self.indicator.HasRun(test, False)
     else:
       # No difference yet and less than three runs -> add another run and
       # remember the output for comparison.
       test.run += 1
       test.output = result[1]
       pool.add([TestJob(test)])
@@ skipping 68 matching lines @@
     if self.context.verbose:
       print text
       sys.stdout.flush()
 
 
 class BreakNowException(Exception):
   def __init__(self, value):
     self.value = value
   def __str__(self):
     return repr(self.value)