Add back buttons and load new 'pages' by calling javascript.

build/android/pylib/results/presentation/test_results_presentation.py

 #! /usr/bin/env python
 #
 # Copyright 2017 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 collections
 import json
 import tempfile
@@ skipping 60 matching lines @@
     rowspan: Rowspan HTML attribute.
   """
   return {
     'cell_type': 'links',
     'class': html_class,
     'links': links,
     'rowspan': rowspan,
   }
 
 
+def action_cell(action, data, html_class):

[mikecase (-- gone --), 2017/04/07 23:57:20]

I do not think we should pass javascript functions in as strings.

[BigBossZhiling, 2017/04/10 19:43:51]

Acknowledged.

+  """Formats table cell with javascript actions.
+
+  Args:
+    action: Javscript action.
+    data: Data in cell.
+    class: Class for table cell.
+  """
+  return {
+    'cell_type': 'action',
+    'action': action,
+    'data': data,
+    'class': html_class,
+  }
+
+
 def logs_cell(result):
   """Formats result logs data for processing in jinja template."""
   link_list = []
   for name, href in result.get('links', {}).iteritems():
     link_list.append(link(
         data=name,
         href=href,
         target=LinkTarget.NEW_TAB))
 
   if link_list:
@@ skipping 67 matching lines @@
 
   header_row = [
     cell(data='suite_name', html_class='text'),
     cell(data='number_success_tests', html_class='number'),
     cell(data='number_fail_tests', html_class='number'),
     cell(data='all_tests', html_class='number'),
     cell(data='elapsed_time_ms', html_class='number'),
   ]
 
   footer_row = [
-    links_cell(
-        links=[
-            link(href=('?suite=%s' % 'TOTAL'),
-                 target=LinkTarget.CURRENT_TAB,
-                 data='TOTAL')
-        ],
-    ),             # suite_name
+    action_cell(
+          'showTestsOfOneSuiteOnly(\'TOTAL\')',

[mikecase (-- gone --), 2017/04/10 17:03:06]

nit: use double quotes here instead of \'

[BigBossZhiling, 2017/04/10 19:43:51]

Done.

+          'TOTAL',
+          'center'
+        ),         # TOTAL
     cell(data=0),  # number_success_tests
     cell(data=0),  # number_fail_tests
     cell(data=0),  # all_tests
     cell(data=0),  # elapsed_time_ms
   ]
 
   suite_row_dict = {}
   for test_name, test_results in results_dict.iteritems():
     # TODO(mikecase): This logic doesn't work if there are multiple test runs.
     # That is, if 'per_iteration_data' has multiple entries.
     # Since we only care about the result of the last test run.
     result = test_results[-1]
 
     suite_name = (test_name.split('#')[0] if '#' in test_name
                   else test_name.split('.')[0])
     if suite_name in suite_row_dict:
       suite_row = suite_row_dict[suite_name]
     else:
       suite_row = [
-        links_cell(
-            links=[
-                link(href=('?suite=%s' % suite_name),
-                     target=LinkTarget.CURRENT_TAB,
-                     data=suite_name)],
-            html_class='left'
+        action_cell(
+          'showTestsOfOneSuiteOnly(\'%s\')' % suite_name,

[mikecase (-- gone --), 2017/04/10 17:03:06]

same here

[BigBossZhiling, 2017/04/10 19:43:51]

Done.

+          suite_name,
+          'left'
         ),             # suite_name
         cell(data=0),  # number_success_tests
         cell(data=0),  # number_fail_tests
         cell(data=0),  # all_tests
         cell(data=0),  # elapsed_time_ms
       ]
 
     suite_row_dict[suite_name] = suite_row
 
     suite_row[ALL_COUNT_INDEX]['data'] += 1
@@ skipping 110 matching lines @@
     print upload_to_google_bucket(result_html_string.encode('UTF-8'),
                                   args.test_name, args.builder_name,
                                   args.build_number, args.bucket,
                                   args.server_url, args.content_type)
   else:
     raise IOError('--json-file %s not found.' % args.json_file)
 
 
 if __name__ == '__main__':
   sys.exit(main())