Bug 1333860 - Fix setting e10s property on counter subtests r?jmaher draft
authorWilliam Lachance <wlachance@mozilla.com>
Wed, 25 Jan 2017 16:09:40 -0500
changeset 466847 c5bee8387adb4e60517918664df33b2ae3f6552c
parent 465527 8ff550409e1d1f8b54f6f7f115545dbef857be0b
child 543535 ff42c0a6d74a611c2958378fd39a5e3b56508d56
push id43016
push userwlachance@mozilla.com
push dateThu, 26 Jan 2017 18:26:56 +0000
reviewersjmaher
bugs1333860
milestone54.0a1
Bug 1333860 - Fix setting e10s property on counter subtests r?jmaher MozReview-Commit-ID: H7YZDn1MUYc
testing/talos/talos/output.py
--- a/testing/talos/talos/output.py
+++ b/testing/talos/talos/output.py
@@ -55,22 +55,20 @@ class Output(object):
 
         for test in self.results.results:
             # serialize test results
             tsresult = None
             if not test.using_xperf:
                 subtests = []
                 suite = {
                     'name': test.name(),
-                    'subtests': subtests,
+                    'extraOptions': self.results.extra_options or [],
+                    'subtests': subtests
                 }
 
-                if self.results.extra_options:
-                    suite['extraOptions'] = self.results.extra_options
-
                 suites.append(suite)
                 vals = []
                 replicates = {}
 
                 # TODO: counters!!!! we don't have any, but they suffer the
                 # same
                 for result in test.results:
                     # XXX this will not work for manifests which list
@@ -161,16 +159,17 @@ class Output(object):
                             subtest['value'] = vals[0]
                     else:
                         # calculate mean value
                         if len(vals) > 0:
                             varray = [float(v) for v in vals]
                             subtest['value'] = filter.mean(varray)
             if counter_subtests:
                 suites.append({'name': test.name(),
+                               'extraOptions': self.results.extra_options or [],
                                'subtests': counter_subtests})
         return test_results
 
     def output(self, results, results_url, tbpl_output):
         """output to the a file if results_url starts with file://
         - results : json instance
         - results_url : file:// URL
         """