zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Apply some more hacks.

Browse Source 
Bug: chromium:1029452
Change-Id: I1be9d42b95f78a2f7f72fa304a35f852167c59e0
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/170461
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#30792}
This commit is contained in:
Patrik Höglund
2020-03-13 16:08:08 +01:00
parent 457c8cfd47
commit a89ad6124b
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
tools_webrtc/perf/catapult_uploader.py
View File

@ -41,7 +41,7 @@ def _SendHistogramSet(url, histograms, oauth_token):
""" """
headers = {'Authorization': 'Bearer %s' % oauth_token} headers = {'Authorization': 'Bearer %s' % oauth_token}
serialized = json.dumps(_ApplyAllBinsHack(histograms.AsDicts()), indent=4) serialized = json.dumps(_ApplyHacks(histograms.AsDicts()), indent=4)
if url.startswith('http://localhost'): if url.startswith('http://localhost'):
# The catapult server turns off compression in developer mode. # The catapult server turns off compression in developer mode.
@ -59,10 +59,13 @@ def _SendHistogramSet(url, histograms, oauth_token):
# TODO(https://crbug.com/1029452): HACKHACK # TODO(https://crbug.com/1029452): HACKHACK
# Remove once we set bin bounds correctly in the proto writer. # Remove once we set bin bounds correctly in the proto writer.
def _ApplyAllBinsHack(dicts): def _ApplyHacks(dicts):
for d in dicts: for d in dicts:
if 'name' in d: if 'name' in d:
d['allBins'] = [[1]] d['allBins'] = [[1]]
del d['binBoundaries']
d['diagnostics']['stories']['values'][0] = (
d['diagnostics']['stories']['values'][0].replace('/', '_'))
return dicts return dicts
@ -92,7 +95,7 @@ def _AddBuildInfo(histograms, options):
def _DumpOutput(histograms, output_file): def _DumpOutput(histograms, output_file):
with output_file: with output_file:
json.dump(_ApplyAllBinsHack(histograms.AsDicts()), output_file, indent=4) json.dump(_ApplyHacks(histograms.AsDicts()), output_file, indent=4)
# TODO(https://crbug.com/1029452): Remove this once # TODO(https://crbug.com/1029452): Remove this once