Chromium Code Reviews| Index: tools/perf/core/perf_benchmark.py |
| diff --git a/tools/perf/core/perf_benchmark.py b/tools/perf/core/perf_benchmark.py |
| index 4158e2d23219bf17eea478976e9230368c9bec6d..23211fff050ff3f4cbf43dd9df98479ad437d072 100644 |
| --- a/tools/perf/core/perf_benchmark.py |
| +++ b/tools/perf/core/perf_benchmark.py |
| @@ -24,6 +24,23 @@ class PerfBenchmark(benchmark.Benchmark): |
| """ To be overridden by perf benchmarks. """ |
| pass |
| + def GetExtraIterationInfo(self): |
| + info = {} |
| + |
| + botName = os.environ.get('BUILDBOT_BUILDERNAME') |
| + if botName: |
| + info['botName'] = botName |
|
sullivan
2016/06/10 16:56:57
Do we need the bot name? The dashboard automatical
|
| + |
| + buildNumber = os.environ.get('BUILDBOT_BUILDNUMBER') |
| + if buildNumber: |
| + info['buildNumber'] = buildNumber |
| + |
| + revision = os.environ.get('BUILDBOT_REVISION') |
| + if revision: |
| + info['revision'] = revision |
| + |
|
sullivan
2016/06/10 16:56:57
These are pulled in via the recipe already.
|
| + return info |
| + |
| def CustomizeBrowserOptions(self, options): |
| # Subclass of PerfBenchmark should override SetExtraBrowserOptions to add |
| # more browser options rather than overriding CustomizeBrowserOptions. |