Allow filing a bug from the "Link Bug" dialog

Tools/GardeningServer/ui/ct-builder-grid.html

Index: Tools/GardeningServer/ui/ct-builder-grid.html
diff --git a/Tools/GardeningServer/ui/ct-builder-grid.html b/Tools/GardeningServer/ui/ct-builder-grid.html
index 98694b127d00d40d6296d7e647175f441b81f780..60231091ef0f6d8f2e4ef6ac361c4935fa41f2dc 100644
--- a/Tools/GardeningServer/ui/ct-builder-grid.html
+++ b/Tools/GardeningServer/ui/ct-builder-grid.html
@@ -14,14 +14,14 @@ found in the LICENSE file.
         display: block;
       }
     </style>
-    <template repeat="{{ builder in _builders }}">
+    <template repeat="{{ builder in builders }}">
       <ct-builder builder="{{ builder }}"></ct-builder>
     </template>
   </template>
   <script>
     Polymer({
       failures: null,
-      _builders: null,
+      builders: null,

[ojan, 2014/08/26 03:14:17]

This really wants to be a model class. Can you create a CTBuilderList model
class that takes a list of failures as it's arguments and does all the work
below to create an array of CTBuilder's? Then have ct-failure-card create that
model and pass it down into ct-builder-grid. Then ct-failure-card doesn't need
to reach into ct-builder-grid to get this data.

[cbiesinger, 2014/09/11 06:24:34]

Done in the other CLs you reviewed.

 
       failuresChanged: function() {
         var builderMap = {};
@@ -39,12 +39,12 @@ found in the LICENSE file.
           }).bind(this));
         }, this);
 
-        this._builders = [];
+        this.builders = [];
 
         Object.keys(builderMap).sort().forEach(function(builder) {
           Object.keys(builderMap[builder]).sort().forEach(function(masterUrl) {
             var result = builderMap[builder][masterUrl];
-            this._builders.push(new CTBuilder(masterUrl, builder,
+            this.builders.push(new CTBuilder(masterUrl, builder,
                 result.earliestFailingBuild, result.failingBuildCount));
           }, this);
         }, this);