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

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

Issue 453543002: Add last update time to Sheriff-O-Matic. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: 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/ui/ct-failure-analyzer.html
diff --git a/Tools/GardeningServer/ui/ct-failure-analyzer.html b/Tools/GardeningServer/ui/ct-failure-analyzer.html
index f31bc4af8cf6c55f313cf59c22008843cefd860b..d1465cbf95a381858e05e3f8493aeb165ea1eb01 100644
--- a/Tools/GardeningServer/ui/ct-failure-analyzer.html
+++ b/Tools/GardeningServer/ui/ct-failure-analyzer.html
@@ -6,13 +6,14 @@ found in the LICENSE file.
<link rel="import" href="../model/ct-failure.html">
-<polymer-element name="ct-failure-analyzer" attributes="failures builderLatestRevisions">
+<polymer-element name="ct-failure-analyzer" attributes="failures builderLatestRevisions date">
<script>
// FIXME: Don't use a polymer component for this. Instead use a Failures model
// object that knows how to do the XHR and process the data appropriately.
Polymer({
builderLatestRevisions: null,
failures: null,
+ date: null,
ojan 2014/08/08 06:05:12 Lets call this lastUpdateDate to be more explicit.
teravest 2014/08/08 22:21:27 Done.
// FIXME: Get this from https://chromium.googlesource.com/chromium/tools/build/+/master/scripts/slave/gatekeeper_trees.json?format=text.
_trees: {
@@ -35,6 +36,7 @@ found in the LICENSE file.
net.json('http://auto-sheriff.appspot.com/data').then(function(data) {
// FIXME: Don't special-case the blink master.
this.builderLatestRevisions = data.latest_revisions['chromium.webkit'];
+ this.date = data.date;
ojan 2014/08/08 06:05:12 Lets store an actual Date object instead of a time
teravest 2014/08/08 22:21:27 Done.
// FIXME: Make this a model class intead of a dumb object.
this.failures = {};
data.range_groups.forEach(function(group) {
« no previous file with comments | « no previous file | Tools/GardeningServer/ui/ct-last-updated.html » ('j') | Tools/GardeningServer/ui/ct-last-updated.html » ('J')

Powered by Google App Engine
This is Rietveld 408576698