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

Tools/GardeningServer/model/ct-failure-group.html

Index: Tools/GardeningServer/model/ct-failure-group.html
diff --git a/Tools/GardeningServer/model/ct-failure-group.html b/Tools/GardeningServer/model/ct-failure-group.html
index 812a2a5efaac998e5f0e7996e3e8759402c5198c..b583a109000c2b94d4a16612847511baada9b176 100644
--- a/Tools/GardeningServer/model/ct-failure-group.html
+++ b/Tools/GardeningServer/model/ct-failure-group.html
@@ -14,8 +14,19 @@ function CTFailureGroup(failures, commitList) {
     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,