Commit 6707e4b1 authored by Caleb Rouleau's avatar Caleb Rouleau Committed by Commit Bot

[Benchmarking] Nit: don't convert to list first

Bug: 1030840
Change-Id: I5cc64282e9405935686cbb9b6d249fd4d3d462e4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974393
Auto-Submit: Caleb Rouleau <crouleau@chromium.org>
Reviewed-by: default avatarJohn Chen <johnchen@chromium.org>
Commit-Queue: John Chen <johnchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#726081}
parent 5df983c6
...@@ -44,7 +44,7 @@ class PerfPlatform(object): ...@@ -44,7 +44,7 @@ class PerfPlatform(object):
_IsPlatformSupported(b.benchmark, self._platform_os)]) _IsPlatformSupported(b.benchmark, self._platform_os)])
# pylint: enable=redefined-outer-name # pylint: enable=redefined-outer-name
benchmark_names = [config.name for config in self._benchmark_configs] benchmark_names = [config.name for config in self._benchmark_configs]
assert len(list(set(benchmark_names))) == len(benchmark_names), ( assert len(set(benchmark_names)) == len(benchmark_names), (
'Make sure that a benchmark does not appear twice.') 'Make sure that a benchmark does not appear twice.')
base_file_name = name.replace(' ', '_').lower() base_file_name = name.replace(' ', '_').lower()
......
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