Added different colors for success/failure and code search.

scripts/slave/recipe_modules/chromium_android/resources/test_results_presentation.py

 #! /usr/bin/env python
 # Copyright 2016 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 json
 import logging
 import os
 import sys
 
 # Load jinja2.
 CURRENT_DIR = os.path.dirname(os.path.abspath(__file__))
 BASE_DIR = os.path.abspath(os.path.join(
     CURRENT_DIR, '..', '..', '..', '..', '..'))
 sys.path.append(os.path.join(BASE_DIR, 'third_party', 'markupsafe'))
 sys.path.append(os.path.join(BASE_DIR, 'third_party', 'jinja2'))
 import jinja2
 jinja_environment = jinja2.Environment(
     loader = jinja2.FileSystemLoader(os.path.dirname(__file__)))
 
 # Get result details from json path and then convert results to html.
-def result_details(json_path):
+def result_details(json_path, cs_base_url):
   with open(json_path) as json_file:
     json_object = json.loads(json_file.read())
     results_list = []
     for testsuite_run in json_object['per_iteration_data']:
       for test, test_runs in testsuite_run.iteritems():
         results_list.extend([
             {
                 'name': test,
                 'status': tr['status'],
                 'duration': tr['elapsed_time_ms'],
                 'output_snippet' : tr['output_snippet']
             } for tr in test_runs])
-    return results_to_html(results_list)
+    return results_to_html(results_list, cs_base_url)
 
 # Convert list of test results into html format.
-def results_to_html(results):
+def results_to_html(results, cs_base_url):
+  def code_search(test):
+    search = test.replace('#', '.')
+    url = cs_base_url if cs_base_url else 'https://cs.chromium.org'

[mikecase (-- gone --), 2016/08/12 18:58:52]

Should probably either set 

http://cs.chromium.org 

as the default to the script --cs-base-url arg or default to this function arg.

[BigBossZhiling, 2016/08/12 21:23:03]

Done.

+    return '%s/?q=%s&type=cs' % (url, search)
+
   suite_row_dict = {}
   test_row_list = []
   for result in results:
-    data = [{'data': result['name'], 'class': 'align-left'},
-            {'data': result['status'], 'class': 'align-center'},
-            {'data': result['duration'], 'class': 'align-center'},
-            {'data': result['output_snippet'], 'class': 'align-left is-pre'}]
+    # Constructing test_row_list.
+    data = [{'data': result['name'], 'class': 'left',
+             'link': code_search(result['name'])},
+            {'data': result['status'], 
+             'class': 'center ' + result['status'].lower()},
+            {'data': result['duration'], 'class': 'center'},
+            {'data': result['output_snippet'], 
+             'class': 'left', 'is_pre': True}]
 
     test_row_list.append(data)
-    suite_name = result['name'][:result['name'].index('#')]
 
+    # Constructing suite_row_dict
+    test_case_path = result['name']
+    suite_name = test_case_path[:test_case_path.index('#')]

[mikecase (-- gone --), 2016/08/12 18:58:52]

super nit:

I usually this this done by...

test_case_path.split('#')[0]

and I think it makes it clearer.

[BigBossZhiling, 2016/08/12 21:23:03]

Done.

     # 'suite_row' is [name, success_count, fail_count, all_count, time].
     SUCCESS_COUNT = 1
     FAIL_COUNT = 2
     ALL_COUNT = 3
     TIME = 4
 
     if suite_name in suite_row_dict:
       suite_row = suite_row_dict[suite_name]  
     else:
-      suite_row = [{'data': suite_name, 'class' : 'align-left'},
-                   {'data': 0, 'class': 'align-center'},
-                   {'data': 0, 'class': 'align-center'},
-                   {'data': 0, 'class': 'align-center'},
-                   {'data': 0, 'class': 'align-center'}]
+      suite_row = [{'data': suite_name, 'class' : 'left'},
+                   {'data': 0, 'class': 'center'},
+                   {'data': 0, 'class': 'center'},
+                   {'data': 0, 'class': 'center'},
+                   {'data': 0, 'class': 'center'}]
       suite_row_dict[suite_name] = suite_row
 
     suite_row[ALL_COUNT]['data'] += 1
     if result['status'] == 'SUCCESS':
       suite_row[SUCCESS_COUNT]['data'] += 1
     elif result['status'] == 'FAILURE':
       suite_row[FAIL_COUNT]['data'] += 1
     suite_row[TIME]['data'] += result['duration']
 
+  for suite in suite_row_dict.values():
+    if suite[FAIL_COUNT]['data'] > 0:
+      suite[FAIL_COUNT]['class'] += ' failure'
+    else:
+      suite[FAIL_COUNT]['class'] += ' success'
+
   test_table_values = {
     'table_id' : 'test_table',
     'table_headers' : [('text', 'test_name'),
                        ('text', 'status'),
                        ('number', 'duration'),
                        ('text', 'output_snippet'),
                       ],
     'table_rows' : test_row_list,
   }
 
@@ skipping 12 matching lines @@
       os.path.join('template', 'main.html'))
   return main_template.render(
       {'tb_values': [suite_table_values, test_table_values]})
 
 def main():
   logging.basicConfig(level=logging.INFO)
   parser = argparse.ArgumentParser()
   parser.add_argument('--json-file', help='Path of json file.', required=True)
   parser.add_argument('--html-file', help='Path to store html file.',
                       required=True)
+  parser.add_argument('--cs-base-url', help='Base url for code search.')

[mikecase (-- gone --), 2016/08/12 18:58:52]

consider adding cs.chromium.org the default arg value.

[BigBossZhiling, 2016/08/12 21:23:03]

Done.

+
   args = parser.parse_args()
   if os.path.exists(args.json_file):
-    result_html_string = result_details(args.json_file)
+    result_html_string = result_details(args.json_file, args.cs_base_url)
 
     with open(args.html_file, 'w') as html:
       html.write(result_html_string)
   else:
-    raise exception('Json file of result details is not found.')
+    raise Exception('Json file of result details is not found.')
 
 if __name__ == '__main__':
   sys.exit(main())