diff options
| author | cmtice <cmtice@google.com> | 2014-03-04 10:46:38 -0800 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-03-06 20:15:35 +0000 |
| commit | e77818d606f46e84a592702272d73715b321a773 (patch) | |
| tree | 6c7e1b039849fbbe2b68c5d9c1c649d5777a3faf /crosperf/benchmark_run.py | |
| parent | 32f09363f80eb73ba1a22bd90d6e609e8f991c8f (diff) | |
| download | toolchain-utils-e77818d606f46e84a592702272d73715b321a773.tar.gz | |
Fix bug introduced with the --show_all_results flag, using cache.
The call to self._ProcessResults is supposed to take the show_all_results
flag as an argument. This was overlooked in the read-it-out-of-the-cache
path when this change was first introduced. This CL fixes that.
BUG=None
TEST=I tested cached and non-cached runs, with and without the
--show_all_results option. Everything worked as it should.
Change-Id: Ibf8f206406b0a4535d48b598709e62135289f993
Reviewed-on: https://chrome-internal-review.googlesource.com/156367
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Commit-Queue: Caroline Tice <cmtice@google.com>
Diffstat (limited to 'crosperf/benchmark_run.py')
| -rw-r--r-- | crosperf/benchmark_run.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/crosperf/benchmark_run.py b/crosperf/benchmark_run.py index d06f2bf7..fa40997c 100644 --- a/crosperf/benchmark_run.py +++ b/crosperf/benchmark_run.py @@ -76,7 +76,8 @@ class BenchmarkRun(threading.Thread): self._logger, self.label, self.share_users, - self.benchmark.suite + self.benchmark.suite, + self.benchmark.show_all_results ) self.result = self.cache.ReadResult() @@ -215,7 +216,8 @@ class MockBenchmarkRun(BenchmarkRun): self._logger, self.label, self.share_users, - self.benchmark.suite + self.benchmark.suite, + self.benchmark.show_all_results ) self.result = self.cache.ReadResult() |