Updating the SwarmingIsolatedScriptTest to upload chartjson results to the

scripts/slave/results_dashboard.py

 #!/usr/bin/env python
 # Copyright (c) 2013 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.
 
 """Functions for adding results to perf dashboard."""
 
 import calendar
 import datetime
 import httplib
@@ skipping 218 matching lines @@
     bot: A string which comes from perf_id, e.g. linux-release.
     buildername: Builder name (for stdio links).
     buildnumber: Build number (for stdio links).
     supplemental_dict: A dictionary of extra data to send with a point;
         this includes revisions and annotation data.
     is_ref: True if this is a reference build, False otherwise.
 
   Returns:
     A dictionary in the format accepted by the perf dashboard.
   """
+  print "1"

[Ken Russell (switch to Gerrit), 2016/09/30 21:45:32]

Remove debugging prints -- unless you really meant to leave them in in
production, in which case add TODOs about removing them.

[eyaich1, 2016/10/03 13:32:56]

Done.

   if not chart_json:
     print 'Error: No json output from telemetry.'
     print '@@@STEP_FAILURE@@@'
 
   # The master name used for the dashboard is the CamelCase name returned by
   # GetActiveMaster(), and not the canonical master name with dots.
   master = slave_utils.GetActiveMaster()
   point_id, versions = _RevisionNumberColumns(revision_dict, prefix='')
 
+  print "2"
   supplemental = {}
   for key in supplemental_dict:
     if key.startswith('r_'):
       versions[key.replace('r_', '', 1)] = supplemental_dict[key]
     if key.startswith('a_'):
       supplemental[key.replace('a_', '', 1)] = supplemental_dict[key]
 
+  print "3"
   supplemental.update(
       _GetStdioUriColumn(test_name, buildername, buildnumber))
 
   # TODO(sullivan): The android recipe sends "test_name.reference"
   # while the desktop one just sends "test_name" for ref builds. Need
   # to figure out why.
   # https://github.com/catapult-project/catapult/issues/2046
   test_name = test_name.replace('.reference', '')
 
+  print "4"
   fields = {
       'master': master,
       'bot': bot,
       'test_suite_name': test_name,
       'point_id': point_id,
       'supplemental': supplemental,
       'versions': versions,
       'chart_data': chart_json,
       'is_ref': is_ref,
   }
@@ skipping 152 matching lines @@
     master, bot, test, revision = (
         data[0]['master'], data[0]['bot'], data[0]['test'], data[0]['revision'])
   else:
     master, bot, test, revision = (
         data['master'], data['bot'], data['chart_data']['benchmark_name'],
         data['point_id'])
   results_link = url + RESULTS_LINK_PATH % (
       urllib.quote(master), urllib.quote(bot), urllib.quote(test.split('/')[0]),
       revision)
   return '@@@STEP_LINK@%s@%s@@@' % ('Results Dashboard', results_link)