[sheriff-o-matic]: Basic trooper display for sheriff-o-matic.

Tools/GardeningServer/ui/ct-failure-card.html

 <!--
 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-builder-grid.html">
-<link rel="import" href="ct-commit-list.html">
-<link rel="import" href="ct-test-list.html">
+<link rel="import" href="ct-sheriff-card.html">
+<link rel="import" href="ct-cq-latency-card.html">
+<link rel="import" href="ct-tree-status-card.html">
+<link rel="import" href="ct-cycle-time-card.html">
 <link rel="import" href="../bower_components/paper-dialog/paper-dialog.html">
 <link rel="import" href="../bower_components/paper-dialog/paper-dialog-transition.html">
 <link rel="import" href="../bower_components/paper-input/paper-input.html">
 
 <polymer-element name="ct-failure-card" attributes="group commitLog tree bug">
   <template>
     <style>
       :host {
         display: flex;
       }
@@ skipping 20 matching lines @@
         box-shadow: inset 0 1px 2px rgba(0, 0, 0, 0.3);
       }
 
       paper-button:active, paper-button:hover {
         background: #f8f8f8;
         border-color: #c6c6c6;
         box-shadow: 0px 1px 1px rgba(0, 0, 0, 0.1);
         color: #222;
       }
 
-      ct-builder-grid {
-        margin-right: 10px;
-        width: 250px;
-      }
-
-      #failure {
-        flex: 1;
-      }
-
       #buttons {
         display: flex;
         flex-direction: column;
       }
 
-      .snoozed {

[ojan, 2014/09/02 02:35:11]

Let's put the snoozed out here and put it on a div wrapping the cards. That way
we can avoid duplicating the snooze styling and class in each card.

[shans, 2014/09/04 01:59:43]

Done.

-        opacity: 0.5;
+      .card {
+        flex: 1;
       }
     </style>
-    <ct-builder-grid failures="{{ group.failures }}"></ct-builder-grid>
-    <div id="failure" class="{{ { snoozed: group.isSnoozed } | tokenList }}">
-      <ct-test-list tests="{{ group.failures }}" tree="{{ tree }}"></ct-test-list>
-      <ct-commit-list commitList="{{ group.commitList }}"></ct-commit-list>
-    </div>
+    <template if="{{ group.type == 'sheriff' }}">

[ojan, 2014/09/02 02:35:11]

Instead of ct-failure-card containing different types of cards, I was picturing
that the different card would be a sibling to ct-failure-card and then each card
type would include the buttons via composition. That way, the failure card isn't
littered with stuff that doesn't make sense for it (e.g. ct-cq-latency-card
shouldn't know anything about commitLists).

[shans, 2014/09/04 01:59:43]

I think the spirit of this is done now (ct-trooper-card doesn't know anything
about commitLists, neither sub-card type knows about annotations as that's
handled at this level).

+      <ct-sheriff-card class='card' group="{{ group }}"></ct-sheriff-card>
+    </template>
+    <template if="{{ group.type == 'cq_latency' }}">
+      <ct-cq-latency-card class='card' group="{{ group }}"></ct-cq-latency-card>
+    </template>
+    <template if="{{ group.type == 'tree_status' }}">
+      <ct-tree-status-card class='card' group="{{ group }}"></ct-tree-status-card>
+    </template>
+    <template if="{{ group.type == 'cycle_time' }}">
+      <ct-cycle-time-card class='card' group="{{ group }}"></ct-cycle-time-card>
+    </template>
     <div id="buttons">
       <paper-button id="examine" on-tap="{{ examine }}" label="Examine"></paper-button>
       <template if="{{ !group.isSnoozed }}">
         <paper-button id="snooze" on-tap="{{ snooze }}" label="Snooze"></paper-button>
       </template>
       <template if="{{ group.isSnoozed }}">
         <paper-button id="snooze" on-tap="{{ unsnooze }}" label="Unsnooze"></paper-button>
       </template>
       <paper-button id="link-bug" on-tap="{{ linkBug }}" label="Link Bug"></paper-button>
       <template if="{{ group.bug !== undefined }}">
@@ skipping 27 matching lines @@
 
       snooze: function() {
         this.group.snoozeUntil(Date.now() + 60 * 60 * 1000);
       },
 
       unsnooze: function() {
         this.group.unsnooze();
       },
 
       _updateCommitList: function() {
-        if (this.group && this.commitLog)
+        if (this.group && this.group.commitList && this.commitLog)
           this.group.commitList.update(this.commitLog);
       },
 
       linkBug: function() {
         this.$.bug.value = this.group.bug;
         this.$.bugDialog.toggle();
       },
 
       saveBug: function() {
         this.group.setBug(this.$.bug.value);
       },
 
       removeBug: function() {
         this.group.clearBug();
       },
     });
   </script>
 </polymer-element>