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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_from_try_jobs.py

Issue 2112133002: Greatly simplify buildbot module and remove usage of master name. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased 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
Index: third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_from_try_jobs.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_from_try_jobs.py b/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_from_try_jobs.py
index 10fbb043771def52d23781b337f47b2f6ed5eeb9..879285b505af4451f8708c2148a59a1a4f21f50b 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_from_try_jobs.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_from_try_jobs.py
@@ -38,8 +38,8 @@ class RebaselineFromTryJobs(AbstractParallelRebaselineCommand):
def _unexpected_mismatch_results(self, try_job):
- results_url = self._results_url(try_job.builder_name, try_job.master_name, try_job.build_number)
- builder = self._tool.buildbot.builder_with_name(try_job.builder_name, try_job.master_name)
+ results_url = self._results_url(try_job.builder_name, try_job.build_number)
+ builder = self._tool.buildbot.builder_with_name(try_job.builder_name)
layout_test_results = builder.fetch_layout_test_results(results_url)
if layout_test_results is None:
_log.warning('Failed to request layout test results from "%s".', results_url)
@@ -62,7 +62,6 @@ class RebaselineFromTryJobs(AbstractParallelRebaselineCommand):
for job in jobs:
_log.info(' Builder: %s', job.builder_name)
- _log.info(' Master: %s', job.master_name)
_log.info(' Build: %s', job.build_number)
test_results = self._unexpected_mismatch_results(job)
if test_results:

Powered by Google App Engine
This is Rietveld 408576698