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

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

Issue 349783003: Polymer-based Garden-O-Matic should have a basic results panel (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix UI nit Created 6 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: Tools/GardeningServer/ui/ct-failure-card.html
diff --git a/Tools/GardeningServer/ui/ct-failure-card.html b/Tools/GardeningServer/ui/ct-failure-card.html
new file mode 100644
index 0000000000000000000000000000000000000000..9007a6b21297719669883026c862b987306d3aa7
--- /dev/null
+++ b/Tools/GardeningServer/ui/ct-failure-card.html
@@ -0,0 +1,50 @@
+<!--
+Copyright 2014 The Chromium Authors. All rights reserved.
+Use of this source code is governed by a BSD-style license that can be
+found in the LICENSE file.
+-->
+
+<link rel="import" href="ct-commit-list.html">
+<link rel="import" href="ct-test-list.html">
+
+<polymer-element name="ct-failure-card" attributes="failures">
+ <template>
+ <style>
+ :host {
+ display: block;
+ margin: 5px;
+ padding: 5px;
+ border-top: 1px solid lightgrey;
+ min-height: 30px;
+ }
+
+ :host(:hover) > #examine {
+ display: initial;
+ }
+
+ #examine {
+ display: none;
+ float: right;
+ }
+
+ paper-button {
+ background: #4285f4;
+ color: #fff;
+ }
+ </style>
+ <paper-button id="examine" on-tap="{{examine}}">Examine</paper-button>
+ <ct-test-list tests="{{failures|testNames}}"></ct-test-list>
+ <ct-commit-list first="{{failures[0].newestPassingRevision + 1}}"
+ last="{{failures[0].oldestFailingRevision}}"></ct-commits>
michaelpg 2014/06/29 23:16:05 </ct-commit-list>
michaelpg 2014/06/29 23:58:32 first and last are reversed (or, they need to be r
abarth-chromium 2014/06/30 00:43:28 Why are they reversed? The first (earliest) commi
michaelpg 2014/06/30 02:32:38 Sorry, you're totally right. What I was actually s
+ </template>
+ <script>
+ Polymer({
+ testNames: function(failures) {
+ return failures.map(function(failureAnalysis) { return failureAnalysis.testName });
+ },
+ examine: function() {
+ this.fire('ct-examine-failures', this.failures);
+ },
+ });
+ </script>
+</polymer-element>

Powered by Google App Engine
This is Rietveld 408576698