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

Unified Diff: Tools/GardeningServer/ui/ct-failure-analyzer.html

Issue 404463003: Move commit list and builder latest revisions up to ct-unexpected-failures (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 5 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/ui/ct-failure-analyzer.html
diff --git a/Tools/GardeningServer/ui/ct-failure-analyzer.html b/Tools/GardeningServer/ui/ct-failure-analyzer.html
index 6920dc01035802707acca4d4efcea6f866476b87..49ddd683043308751b3a88ea24488f6394952f8d 100644
--- a/Tools/GardeningServer/ui/ct-failure-analyzer.html
+++ b/Tools/GardeningServer/ui/ct-failure-analyzer.html
@@ -4,11 +4,12 @@ Use of this source code is governed by a BSD-style license that can be
found in the LICENSE file.
-->
-<polymer-element name="ct-failure-analyzer" attributes="failures status">
+<polymer-element name="ct-failure-analyzer" attributes="failures status builderLatestRevisions">
<script>
Polymer({
failures: {},
pendingUnexpectedFailures: [],
+ builderLatestRevisions: {},
update: function() {
this._updateFailingBuilders();
@@ -21,6 +22,15 @@ found in the LICENSE file.
}).bind(this));
},
+ _updateBuilderLatestRevisions: function() {
+ this.builderLatestRevisions = {};
+ Object.keys(config.builders, function(builder) {
+ this.builderLatestRevisions[builder] = {
+ blink: model.state.resultsByBuilder[builder].blink_revision,
+ };
+ }.bind(this));
+ },
+
_updateUnexpectedFailures: function() {
this.pendingUnexpectedFailures = [];
var numberOfTestsAnalyzed = 0;
@@ -33,6 +43,7 @@ found in the LICENSE file.
}.bind(this)).then(function() {
this.status = 'Done';
this.failures.unexpected = this.pendingUnexpectedFailures;
+ this._updateBuilderLatestRevisions();
}.bind(this));
}.bind(this));
},

Powered by Google App Engine
This is Rietveld 408576698