Move treestatus.js to a model class.

Tools/GardeningServer/ui/ct-tree-status.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="../model/tree-status.html">
+
 <polymer-element name="ct-tree-status">
   <template>
     <style>
       :host {
         display: flex;
       }
 
       :host([status=throttled]) {
         background-color: #fffc6c;
       }
@@ skipping 21 matching lines @@
       publish: {
         project: '',
         message: '',
         status: {
           value: '',
           reflect: true,
         },
       },
 
       _url: function(project) {
-        return treestatus.urlByName(this.project);
+        var treeStatus = new TreeStatus();
+        return treeStatus.urlByName(this.project);
       },
 
       update: function() {

[ojan, 2014/08/06 22:34:17]

Now that we have a proper model class, lets get this component out of the
business of doing network requests. We're trying to move towards a design where
the model is the thing that does the network request. So, ct-unexpected-failures
will create the TreeStatus model instance and pass it in as a property on this
element. Then ct-unexpected-failures won't call update on this element at all.
It'll call update on the model, which will propagate down to here through data
binding magic.

[dsinclair, 2014/08/07 18:09:23]

Done.

         var url = this._url(this.project) + 'current?format=json';
         return net.json(url).then(function(response) {
           this.updateStatus(response);
         }.bind(this));
       },
 
       updateStatus: function(status) {

[ojan, 2014/08/06 22:34:17]

Can you move most of this logic into the model class?

[dsinclair, 2014/08/07 18:09:23]

Done.

         if (status.can_commit_freely) {
           this.message = null;
           this.status = 'open';
           return;
         }
 
         this.message = status.message + ' by ' + status.username;
         var responseLowerCase = status.message.toLowerCase();
         if (responseLowerCase.indexOf('throttled') != -1) {
           this.status = 'throttled';
         } else if (responseLowerCase.indexOf("closed") != -1) {
           this.status = 'closed';
         } else {
           this.status = 'unknown';
         }
       },
     });
   </script>
 </polymer-element>