Commit 7b0adf2e authored by Sadrul Habib Chowdhury's avatar Sadrul Habib Chowdhury Committed by Commit Bot

telemetry: Fix running smoke tests.

Fix running smoke tests after changes in catapult:
  https://chromium-review.googlesource.com/c/catapult/+/1474597
After that change, it is necessary to update the target platforms
for the browser from the benchmark's supported platforms.

BUG=932409

Change-Id: I7cddec057d987e342890d180485f54f927c61f7a
Reviewed-on: https://chromium-review.googlesource.com/c/1481742
Auto-Submit: Sadrul Chowdhury <sadrul@chromium.org>
Reviewed-by: default avatarCaleb Rouleau <crouleau@chromium.org>
Commit-Queue: Caleb Rouleau <crouleau@chromium.org>
Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
Cr-Commit-Position: refs/heads/master@{#634419}
parent 2a49526f
...@@ -235,6 +235,11 @@ def GenerateBenchmarkOptions(benchmark_class): ...@@ -235,6 +235,11 @@ def GenerateBenchmarkOptions(benchmark_class):
# all crashes and hence remove the need to enable logging in actual perf # all crashes and hence remove the need to enable logging in actual perf
# benchmarks. # benchmarks.
options.browser_options.logging_verbosity = 'non-verbose' options.browser_options.logging_verbosity = 'non-verbose'
# TODO(sadrul): Remove the check once catapult roll happens.
# https://crbug.com/932409
if "GetSupportedPlatforms" in dir(benchmark_class):
options.target_platforms = benchmark_class.GetSupportedPlatforms(
benchmark_class.SUPPORTED_PLATFORMS)
return options return options
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment