Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(263)

Unified Diff: tools/perf/core/perf_benchmark.py

Issue 2055063002: Plumb iteration info (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698