V8: Highligh slow tests not marked as slow.

scripts/slave/recipe_modules/v8/api.py

 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import argparse
 import datetime
 import random
 import re
 import urllib
 
@@ skipping 709 matching lines @@
       'Test: %s' % test['name'],
       'Flags: %s' % ' '.join(test['flags']),
       'Command: %s' % test['command'],
       'Duration: %s' % V8Api.format_duration(test['duration']),
     ]
 
   def _update_durations(self, output, presentation):
     # Slowest tests duration summary.
     lines = []
     for test in output['slowest_tests']:
+      suffix = ''
+      if 'marked_slow' in test and not test['marked_slow']:

[tandrii(chromium), 2016/04/14 13:00:46]

if not test.get('marked_slow'):

[Michael Achenbach, 2016/04/14 13:07:56]

That has a different meaning. I don't want a * for branches, where the test
driver doesn't include that field yet.

[tandrii(chromium), 2016/04/14 13:19:11]

you are right. Then this is the one:
if test.get('marked_slow') is False:

so long as you never have {'marked_slow': None}
anyway, this is nit.

[Michael Achenbach, 2016/04/14 13:32:01]

Done.

+        suffix = ' *'
       lines.append(
-          '%s %s' %(V8Api.format_duration(test['duration']), test['name']))
+          '%s %s%s' % (V8Api.format_duration(test['duration']),
+                       test['name'], suffix))
+
     # Slowest tests duration details.
     lines.extend(['', 'Details:', ''])
     for test in output['slowest_tests']:
       lines.extend(self._duration_results_text(test))
     presentation.logs['durations'] = lines
 
   def _get_failure_logs(self, output, failure_factory):
     def all_same(items):
       return all(x == items[0] for x in items)
 
@@ skipping 334 matching lines @@
   def report_culprits(self, culprit_range):
     assert culprit_range
     if len(culprit_range) > 1:
       text = 'Suspecting multiple commits'
     else:
       text = 'Suspecting %s' % culprit_range[0][:8]
 
     step_result = self.m.step(text, cmd=None)
     for culprit in culprit_range:
       step_result.presentation.links[culprit[:8]] = COMMIT_TEMPLATE % culprit