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

Tools/GardeningServer/model/ct-failure-group.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-commit-list.html">
 
 <script>
 function CTFailureGroup(failures, commitList) {
   this.failures = failures;
   this.commitList = commitList;
   this._annotation = CTFailureGroup._mergeAnnotations(failures.map(function(failure) {
     return failure.annotations();
   }).flatten());
   this._computeProperties();
+  this.type = 'sheriff';
 }
 
+function CTTrooperFailureGroup(data, type, tree, annotation) {

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

Rather than using inheritance for this, in other places I've used create
methods. CTFailureGroup.createSheriffGroup and
CTFailureGroup.createTrooperGroup. Actually, even better here would be to just
have one sub-class and use composition for the different behaviors. So,
something like...

function CTFailureGroup(data, type, annotation) {...}

Where data is one of CTTrooperFailureGroupData or CTSheriffFailureGroupData,
which has all the appropriate data and methods (e.g. now CTFailureGroup wouldn't
know anything about commitLists).

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

Done.

+  this.data = data;
+  this._annotation = annotation || {};
+  this._computeProperties();
+  this.type = type;
+  this.tree = tree;
+}
+
+CTTrooperFailureGroup.prototype = CTFailureGroup.prototype;
+
 CTFailureGroup.prototype.snoozeUntil = function(time) {
   return this._annotate({
     snoozeTime: time,
   });
 };
 
 CTFailureGroup.prototype.unsnooze = function() {
   return this._annotate({
     snoozeTime: undefined,
   });
@@ skipping 84 matching lines @@
   }.bind(this));
 };
 
 CTFailureGroup.fetchAnnotations = function() {
   // FIXME: Fetch annotations from frontend.
   var stored = localStorage.CTFailureGroupAnnotations;
   var annotations = stored ? JSON.parse(stored) : {};
   return Promise.resolve(annotations);
 };
 </script>