V8: Highligh slow tests not marked as slow.

scripts/slave/recipe_modules/v8/api.py

Index: scripts/slave/recipe_modules/v8/api.py
diff --git a/scripts/slave/recipe_modules/v8/api.py b/scripts/slave/recipe_modules/v8/api.py
index c3265be0f2c1167d3a38453a4d809a0fbfe95147..70a6b3ebe7206e6613d3c93bba3ff471263bcbd1 100644
--- a/scripts/slave/recipe_modules/v8/api.py
+++ b/scripts/slave/recipe_modules/v8/api.py
@@ -727,8 +727,13 @@ class V8Api(recipe_api.RecipeApi):
     # 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']: