Upload the test results html file to google bucket.

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
+import time
 import os
+import subprocess
 import sys
 
 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'))
 import jinja2  # pylint: disable=import-error
 JINJA_ENVIRONMENT = jinja2.Environment(
     loader=jinja2.FileSystemLoader(os.path.dirname(__file__)),
     autoescape=True)
@@ skipping 246 matching lines @@
 
   if not 'per_iteration_data' in json_object:
     return 'Error: json file missing per_iteration_data.'
 
   results_dict = collections.defaultdict(list)
   for testsuite_run in json_object['per_iteration_data']:
     for test, test_runs in testsuite_run.iteritems():
       results_dict[test].extend(test_runs)
   return results_to_html(results_dict, cs_base_url, master_name)
 
+def upload_to_google_bucket(html, args):
+  with tempfile.NamedTemporaryFile(suffix='.html') as temp_file:
+    temp_file.write(html)
+    dest = 'html/%s/%s_%s_%s.html' % (
+        args.test_name if args.test_name else 'test_name',

[BigBossZhiling, 2017/03/27 22:54:54]

I am not sure whether we want to make test_name, (chrome_public_test_apk) etc.
to be mandatory here. If not, do you think that test_name, builder_name and
builder_number is good default value?

[jbudorick, 2017/03/28 02:15:33]

I think they should all be mandatory.

+        args.builder_name if args.builder_name else 'builder_name',
+        args.build_number if args.build_number else 'builder_number',
+        time.strftime('%Y_%m_%d_T%H_%M_%S'))
+    gsutil_path = os.path.join(BASE_DIR, 'third_party', 'catapult',
+                               'third_party', 'gsutil', 'gsutil.py')
+
+    subprocess.check_call('python %s cp %s gs://%s/%s' % (
+        gsutil_path, temp_file.name, args.bucket, dest), shell=True)
+  return 'https://storage.googleapis.com/%s/%s' % (args.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('--master-name', help='Master name in urls.')
+  parser.add_argument('--bucket', default='chromium-result-details')
+  parser.add_argument('--builder-name', help='Builder name.')
+  parser.add_argument('--build-number', help='Build number.')
+  parser.add_argument('--test-name', help='The name of the test.')
 
   args = parser.parse_args()
   if os.path.exists(args.json_file):
     result_html_string = result_details(args.json_file, args.cs_base_url,
                                         args.master_name)
-    print result_html_string.encode('UTF-8')
+    print upload_to_google_bucket(result_html_string.encode('UTF-8'),
+                                  args)

[jbudorick, 2017/03/28 02:15:33]

This should pass args.test_name, args.builder_name, args.bucket, and
args.build_number separately.

   else:
     raise IOError('--json-file %s not found.' % args.json_file)
 
 
 if __name__ == '__main__':
   sys.exit(main())