Revert of 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):
-  """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 = [
-    action_cell(
-          'showTestsOfOneSuiteOnlyWithNewState("TOTAL")',
-          'TOTAL',
-          'center'
-        ),         # TOTAL
+    links_cell(
+        links=[
+            link(href=('?suite=%s' % 'TOTAL'),
+                 target=LinkTarget.CURRENT_TAB,
+                 data='TOTAL')
+        ],
+    ),             # 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 = {}
   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 = [
-        action_cell(
-          'showTestsOfOneSuiteOnlyWithNewState("%s")' % suite_name,
-          suite_name,
-          'left'
+        links_cell(
+            links=[
+                link(href=('?suite=%s' % suite_name),
+                     target=LinkTarget.CURRENT_TAB,
+                     data=suite_name)],
+            html_class='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 82 matching lines @@
         sys.executable, gsutil_path, '-h', "Content-Type:%s" % content_type,
         'cp', temp_file.name, 'gs://%s/%s' % (bucket, dest)])
 
   return '%s/%s/%s' % (server_url, bucket, dest)
 
 def main():
   parser = argparse.ArgumentParser()
   parser.add_argument('--json-file', help='Path of json file.', required=True)
   parser.add_argument('--cs-base-url', help='Base url for code search.',
                       default='http://cs.chromium.org')
-  parser.add_argument('--bucket', help='Google storage bucket.', required=True)
+  parser.add_argument('--bucket', default='chromium-result-details')
   parser.add_argument('--builder-name', help='Builder name.', required=True)
   parser.add_argument('--build-number', help='Build number.', required=True)
   parser.add_argument('--test-name', help='The name of the test.',
                       required=True)
   parser.add_argument('--server-url', help='The url of the server.',
-                      default='https://storage.cloud.google.com')
+                      default='https://storage.googleapis.com')
   parser.add_argument(
       '--content-type',
       help=('Content type, which is used to determine '
             'whether to download the file, or view in browser.'),
       default='text/html',
       choices=['text/html', 'application/octet-stream'])
 
   args = parser.parse_args()
   if os.path.exists(args.json_file):
     result_html_string = result_details(args.json_file, args.cs_base_url,
                                         args.bucket, args.server_url)
     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())