Tweak compare_codereview.py.

tools/compare_codereview.py

 #!/usr/bin/python2
 
 # Copyright 2014 Google Inc.
 #
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Skia's Chromium Codereview Comparison Script.
 
 This script takes two Codereview URLs, looks at the trybot results for
@@ skipping 318 matching lines @@
     if not all_bots:
         print >> sys.stderr, (
             'Error:  control %s and roll %s have no common trybots.'
             % (list(control), list(roll)))
         return
 
     control_name = '[control %s]' % control_url.split('/')[-1]
     roll_name = '[roll %s]' % roll_url.split('/')[-1]
 
     out = sys.stdout
-    if verbosity > 0:
-        # Print out summary of all of the bots.
-        out.write('%11s %11s %4s %s\n\n' %
-                  ('CONTROL', 'ROLL', 'DIFF', 'BOT'))
-        for bot in sorted(all_bots):
-            if control[bot].status != roll[bot].status:
-                diff = '****'
-            elif (control[bot].status != 'success' or
-                roll[bot].status != 'success'):
-                diff = '....'
-            else:
-                diff = ''
-            out.write('%11s %11s %4s %s\n' % (
-                    control[bot].status, roll[bot].status, diff, bot))
-        out.write('\n')
-        out.flush()
 
     for bot in sorted(all_bots):
         if (roll[bot].status == 'success'):
             if verbosity > 1:
                 printer(0, '==%s==' % bot)
                 printer(1, 'OK')
             continue
         printer(0, '==%s==' % bot)
 
         for (status, name, url) in (
             (control[bot].status, control_name, control[bot].url),
             (roll[bot].status, roll_name, roll[bot].url)):
 
             if status == 'failure':
                 printer(1, name)
                 results = BuilderHTMLParser.parse(url)
                 for result in results:
                     formatted_result = re.sub(
                         r'(\S*\.html) ', '\n__\g<1>\n', result.text)
+                    # Strip runtimes.
+                    formatted_result = re.sub(r'\(.*\)', '', formatted_result)
                     printer(2, formatted_result)
                     if ('compile' in result.text
                         or '...and more' in result.text):
                         printer(3, re.sub('/[^/]*$', '/', url) + result.url)
             else:
                 printer(1, name)
                 printer(2, status)
         out.write('\n')
 
+    if verbosity > 0:
+        # Print out summary of all of the bots.
+        out.write('%11s %11s %4s %s\n\n' %
+                  ('CONTROL', 'ROLL', 'DIFF', 'BOT'))
+        for bot in sorted(all_bots):
+            if roll[bot].status == 'success':
+                diff = ''
+            elif (control[bot].status == 'success' and
+                     roll[bot].status == 'failure'):
+                diff = '!!!!'
+            elif ('pending' in control[bot].status or
+                  'pending' in roll[bot].status):
+                diff = '....'
+            else:
+                diff = '****'
+            out.write('%11s %11s %4s %s\n' % (
+                    control[bot].status, roll[bot].status, diff, bot))
+        out.write('\n')
+        out.flush()
 
 if __name__ == '__main__':
     if len(sys.argv) < 3:
         print >> sys.stderr, __doc__
         exit(1)
     main(sys.argv[1], sys.argv[2],
          int(os.environ.get('COMPARE_CODEREVIEW_VERBOSITY', 1)))