Make results_dashboard send just one request per test run.

scripts/slave/runtest.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 """A tool to run a chrome test executable, used by the buildbot slaves.
 
 When this is run, the current directory (cwd) should be the outer build
 directory (e.g., chrome-release/build/).
 
@@ skipping 556 matching lines @@
     with file(supplemental_columns_file, 'r') as f:
       supplemental_columns = json.loads(f.read())
   return supplemental_columns
 
 
 def _SendResultsToDashboard(results_tracker, system, test, url, build_dir,
                             mastername, buildername, buildnumber,
                             supplemental_columns_file, extra_columns=None):
   """Sends results from a results tracker (aka log parser) to the dashboard.
 
-  TODO(qyearsley): Change this function and results_dashboard.SendResults so
-  that only one request is made per test run (instead of one per graph name).
-  Also, maybe refactor this function to take fewer arguments.
-
   Args:
     results_tracker: An instance of a log parser class, which has been used to
         process the test output, so it contains the test results.
     system: A string such as 'linux-release', which comes from perf_id.
     test: Test "suite" name string.
     url: Dashboard URL.
     build_dir: Build dir name (used for cache file by results_dashboard).
     mastername: Buildbot master name, e.g. 'chromium.perf'.
         WARNING! This is incorrectly called "masterid" in some parts of the
         dashboard code.
     buildername: Builder name, e.g. 'Linux QA Perf (1)'
     buildnumber: Build number (as a string).
     supplemental_columns_file: Filename for JSON supplemental columns file.
     extra_columns: A dict of extra values to add to the supplemental columns
         dict.
   """
   if system is None:
     # perf_id not specified in factory-properties
     return
   supplemental_columns = _GetSupplementalColumns(build_dir,
                                                  supplemental_columns_file)
   if extra_columns:
     supplemental_columns.update(extra_columns)
-  for logname, log in results_tracker.PerformanceLogs().iteritems():
-    lines = [str(l).rstrip() for l in log]
-    try:
-      results_dashboard.SendResults(logname, lines, system, test, url,
-                                    mastername, buildername, buildnumber,
-                                    build_dir, supplemental_columns)
-    except NotImplementedError as e:
-      print 'Did not submit to results dashboard: %s' % e
+
+  logs_dict = results_tracker.PerformanceLogs()
+  results_dashboard.SendResults(logs_dict, system, test, url, mastername,
+                                buildername, buildername, buildnumber,
+                                build_dir, supplemental_columns)
 
 
 def _BuildCoverageGtestExclusions(options, args):
   gtest_exclusions = {
     'win32': {
       'browser_tests': (
         'ChromeNotifierDelegateBrowserTest.ClickTest',
         'ChromeNotifierDelegateBrowserTest.ButtonClickTest',
         'SyncFileSystemApiTest.GetFileStatuses',
         'SyncFileSystemApiTest.WriteFileThenGetUsage',
@@ skipping 1119 matching lines @@
   finally:
     if did_launch_dbus:
       # It looks like the command line argument --exit-with-session
       # isn't working to clean up the spawned dbus-daemon. Kill it
       # manually.
       _ShutdownDBus()
 
 
 if '__main__' == __name__:
   sys.exit(main())