Add a "Link Bug" button to associate a bug with a failure

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="../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 commits tree">
+<polymer-element name="ct-failure-card" attributes="group commits tree bug">
   <template>
     <style>
       :host {
         display: flex;
       }
 
       /* FIXME: All this paper-button styling should go in a cr-button component so that
          we can use buttons in different places and have them all look the same. */
       paper-button {
         -webkit-user-select: none;
@@ skipping 15 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 {
+      #builder-column {
         margin-right: 10px;
         width: 250px;
       }
 
       #failure {
         flex: 1;
       }
 
       #buttons {
         display: flex;
         flex-direction: column;
       }
 
       .snoozed {
         opacity: 0.5;
       }
     </style>
-    <ct-builder-grid failures="{{ group.failures }}"></ct-builder-grid>
+    <div id="builder-column">

[ojan, 2014/08/15 04:53:39]

This is leftover from a previous iteration of the code, yes? No longer needed?

[cbiesinger, 2014/08/16 00:46:15]

Yes, changed back.

+      <ct-builder-grid failures="{{ group.failures }}"></ct-builder-grid>
+    </div>
     <div id="failure" class="{{ { snoozed: group.isSnoozed } | tokenList }}">
       <ct-test-list tests="{{ group.failures }}" tree="{{ tree }}"></ct-test-list>
       <ct-commit-list first="{{ group.failures[0].lastPassingRevisions }}"
           last="{{ group.failures[0].firstFailingRevisions }}"
           commits="{{ commits }}"></ct-commit-list>
     </div>
     <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.hasBug }}">
+        <div>
+          <a href="http://crbug.com/{{ group.annotation.bugNumber }}">
+            Bug {{ group.annotation.bugNumber }}</a>
+        </div>
+      </template>
     </div>
+
+    <paper-dialog heading="Enter bug number" transition="paper-transition-center" id="bugDialog">
+      <paper-input label="Bug#" type="number" floatingLabel min="10000" id="bugNumber"></paper-input>

[ojan, 2014/08/15 04:53:39]

Why have a min?

[cbiesinger, 2014/08/16 00:46:15]

To make sure that people use stuff that looks like a bug number, but per your
other comment, I'll make this use a string anyway so that URLs work.

+      <paper-button label="OK" on-tap="{{ saveBug }}" affirmative autofocus id="dialogOK"></paper-button>

[ojan, 2014/08/15 04:53:39]

Nit: I prefer the dialogOk style to avoid abominations like XMLHttpRequest.

[ojan, 2014/08/15 04:53:40]

Nit: I prefer the dialogOk style to avoid abominations like XMLHttpRequest.

[cbiesinger, 2014/08/16 00:46:15]

Done.

+    </paper-dialog>
   </template>
   <script>
     Polymer({
       group: null,
       commits: {},
       tree: '',
 
       examine: function() {
         this.fire('ct-examine-failures', this.group);
       },
 
       snooze: function() {
         this.group.snoozeUntil(Date.now() + 60 * 60 * 1000);
       },
 
       unsnooze: function() {
         this.group.unsnooze();
       },
+
+      linkBug: function() {
+        this.$.bugNumber.value = this.group.annotation.bugNumber;
+        this.$.bugDialog.toggle();
+      },
+
+      saveBug: function() {
+        this.group.setBug(this.$.bugNumber.value);
+      },
     });
   </script>
 </polymer-element>