Move builderList creation to ct-failure-card

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">
@@ skipping 47 matching lines @@
 
       #buttons {
         display: flex;
         flex-direction: column;
       }
 
       .snoozed {
         opacity: 0.5;
       }
     </style>
-    <ct-builder-grid failures="{{ group.failures }}"></ct-builder-grid>
+    <ct-builder-grid builderList="{{ _builderList }}"></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>
     <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 }}">
@@ skipping 11 matching lines @@
     <paper-dialog heading="Enter bug number" transition="paper-transition-center" id="bugDialog">
       <paper-input label="Bug# or URL" floatingLabel autofocus id="bug"></paper-input>
       <paper-button label="Remove bug link" on-tap="{{ removeBug }}" dismissive id="dialogRemoveBug"></paper-button>
       <paper-button label="OK" on-tap="{{ saveBug }}" affirmative id="dialogOk"></paper-button>
     </paper-dialog>
   </template>
   <script>
     Polymer({
       group: null,
       commitLog: null,
+      builderList: null,
       tree: '',
 
       observe: {
         group: '_updateCommitList',
-        commitLog: '_updateCommitList'
+        commitLog: '_updateCommitList',
+        'group.failures': '_updateBuilderList',

[ojan, 2014/09/05 01:18:04]

Woah. Didn't realize you could do this.

[cbiesinger, 2014/09/06 00:25:56]

Yeah, also $.foo.bar for polymer elements in your child tree. See
http://www.polymer-project.org/docs/polymer/polymer.html#change-watchers

       },
 
       examine: function() {
         this.fire('ct-examine-failures', this.group);
       },
 
       snooze: function() {
         this.group.snoozeUntil(Date.now() + 60 * 60 * 1000);
       },
 
       unsnooze: function() {
         this.group.unsnooze();
       },
 
       _updateCommitList: function() {
         if (this.group && this.commitLog)
           this.group.commitList.update(this.commitLog);
       },
 
+      _updateBuilderList: function() {
+        this._builderList = new CTBuilderList(this.group.failures);

[ojan, 2014/09/05 01:18:04]

I think we still have a bug where we won't update the builder list if it changes
within a given CTFailure, but that's not new in this patch. But, maybe add a
FIXME? This won't really show up until jyasskin lands his patch making us do
incremental updates (today or tomorrow).

[cbiesinger, 2014/09/06 00:25:56]

Oy, that's true, but I have no idea how to fix that? I'll add a FIXME.

+      },
+
       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>