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

Unified Diff: Tools/GardeningServer/model/ct-failure-group.html

Issue 498523002: [Sheriff-o-matic] Use likely_revisions instead of first_failing/last_passing (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: sort git hashes Created 6 years, 4 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: Tools/GardeningServer/model/ct-failure-group.html
diff --git a/Tools/GardeningServer/model/ct-failure-group.html b/Tools/GardeningServer/model/ct-failure-group.html
index 7eae058130ed966242df0dd826d4a09f2d8e1489..2de1de15c0744e28f662507f4e9c7273f5e87f8e 100644
--- a/Tools/GardeningServer/model/ct-failure-group.html
+++ b/Tools/GardeningServer/model/ct-failure-group.html
@@ -7,9 +7,10 @@ found in the LICENSE file.
<link rel="import" href="ct-commit-list.html">
<script>
-function CTFailureGroup(key, failures, annotation) {
+function CTFailureGroup(key, failures, revisions, annotation) {
this.key = key;
this.failures = failures;
+ this.revisions = revisions.slice() || [];
ojan 2014/08/23 01:23:05 No need for the || here, right?
Mathieu 2014/08/24 23:42:11 Done.
this._annotation = annotation || {};
this._computeProperties();
}
@@ -27,7 +28,7 @@ CTFailureGroup.prototype.unsnooze = function() {
};
CTFailureGroup.prototype.commitList = function(commits) {
- return new CTCommitList(this, commits);
+ return new CTCommitList(commits, this.revisions);
};
CTFailureGroup.prototype.setBug = function(bug) {

Powered by Google App Engine
This is Rietveld 408576698