Bug 1465659 - Take PERFHERDER_EXTRA_OPTIONS into account for static initializer count report. r?build draft
authorMike Hommey <mh+mozilla@glandium.org>
Thu, 31 May 2018 12:55:38 +0900
changeset 801961 5b2994262dd88ce465b2d4258528dbcdae8e78d6
parent 801944 f32597fe6cbef0b076effd74e04591da4b34ac25
child 801962 e6e6fc6d754dbf8f156dd114751e4d1ede91dba2
push id111792
push userbmo:mh+mozilla@glandium.org
push dateThu, 31 May 2018 04:48:26 +0000
reviewersbuild
bugs1465659
milestone62.0a1
Bug 1465659 - Take PERFHERDER_EXTRA_OPTIONS into account for static initializer count report. r?build
toolkit/crashreporter/tools/symbolstore.py
--- a/toolkit/crashreporter/tools/symbolstore.py
+++ b/toolkit/crashreporter/tools/symbolstore.py
@@ -628,16 +628,21 @@ class Dumper:
                     "subtests": [{
                         "name": "num_static_constructors",
                         "value": ctors,
                         "alertChangeType": "absolute",
                         "alertThreshold": 3
                     }]}
                 ]
             }
+            for opt in os.environ.get('PERFHERDER_EXTRA_OPTIONS', '').split():
+                for suite in perfherder_data['suites']:
+                    if opt not in suite.get('extraOptions', []):
+                        suite.setdefault('extraOptions', []).append(opt)
+
             print('PERFHERDER_DATA: %s' % json.dumps(perfherder_data),
                   file=sys.stderr)
 
         elapsed = time.time() - t_start
         print('Finished processing %s in %.2fs' % (file, elapsed),
               file=sys.stderr)
 
 # Platform-specific subclasses.  For the most part, these just have