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

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

Issue 2692423005: rebaseline-cl: Get latest try jobs using git-cl when no --issue given. (Closed)
Patch Set: Rebased Created 3 years, 10 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_cl.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py b/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py
index 8a1401aaa9297ec1843e190a4532f5f277f3fc20..e536cea34a95495bb299a749c333c48ef475c30e 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py
@@ -64,9 +64,11 @@ class RebaselineCL(AbstractParallelRebaselineCommand):
if not issue_number:
return 1
- # TODO(qyearsley): Replace this with git cl try-results to remove
- # dependency on Rietveld. See crbug.com/671684.
- builds = self.rietveld.latest_try_jobs(issue_number, self._try_bots())
+ # TODO(qyearsley): Remove dependency on Rietveld. See crbug.com/671684.
+ if options.issue:
+ builds = self.rietveld.latest_try_jobs(issue_number, self._try_bots())
+ else:
+ builds = self.git_cl().latest_try_jobs(self._try_bots())
if options.trigger_jobs:
if self.trigger_jobs_for_missing_builds(builds):

Powered by Google App Engine
This is Rietveld 408576698