[Sheriff-o-matic] Remove race condition on the commit list.

Tools/GardeningServer/ui/ct-unexpected-failures.html

Index: Tools/GardeningServer/ui/ct-unexpected-failures.html
diff --git a/Tools/GardeningServer/ui/ct-unexpected-failures.html b/Tools/GardeningServer/ui/ct-unexpected-failures.html
index 2be627ddb52e5d4dd9c87d6c193e65386a7eb10c..4f03f1972bc77a0fb2dc7d1601004bd4db61393d 100644
--- a/Tools/GardeningServer/ui/ct-unexpected-failures.html
+++ b/Tools/GardeningServer/ui/ct-unexpected-failures.html
@@ -40,14 +40,14 @@ found in the LICENSE file.
     </style>
     <ct-failure-analyzer id="analyzer" failures="{{ failures }}" builderLatestRevisions="{{ builderLatestRevisions }}" lastUpdateDate="{{ lastUpdateDate }}"></ct-failure-analyzer>
     <div class="toolbar">
-      <ct-revision-details id="revisionDetails" builderLatestRevisions="{{ builderLatestRevisions }}" revisionLog="{{ revisionLog }}" tree="{{ tree }}"></ct-revision-details>
+      <ct-revision-details id="revisionDetails" builderLatestRevisions="{{ builderLatestRevisions }}" commitLog="{{ commitLog }}" tree="{{ tree }}"></ct-revision-details>
       <a href="https://code.google.com/p/chromium/wiki/UsefulURLs">Useful URLs</a>
     </div>
     <ct-tree-status status="{{ treeStatuses['chromium'] }}" state="{{ treeStatuses['chromium'].status }}"></ct-tree-status>
     <ct-tree-status status="{{ treeStatuses['blink'] }}" state="{{ treeStatuses['blink'].status }}"></ct-tree-status>
     <ct-party-time failures="{{ failures }}" tree="{{ tree }}"></ct-party-time>
-    <ct-failure-stream category="default" groups="{{ failures[tree] }}" commits="{{ revisionLog }}" tree="{{ tree }}"></ct-failure-stream>
-    <ct-failure-stream category="snoozed" groups="{{ failures[tree] }}" commits="{{ revisionLog }}" tree="{{ tree }}"></ct-failure-stream>
+    <ct-failure-stream category="default" groups="{{ failures[tree] }}" commitLog="{{ commitLog }}" tree="{{ tree }}"></ct-failure-stream>
+    <ct-failure-stream category="snoozed" groups="{{ failures[tree] }}" commitLog="{{ commitLog }}" tree="{{ tree }}"></ct-failure-stream>
   </template>
   <script>
   (function() {
@@ -57,7 +57,8 @@ found in the LICENSE file.
       tree: '',
 
       created: function() {
-        this.revisionLog = new CTCommitLog(),
+        window.console.log('created CTCommitLog');

[ojan, 2014/08/20 18:47:51]

Leftover debugging code?

[Mathieu, 2014/08/21 14:08:21]

Done.

+        this.commitLog = new CTCommitLog(),
         this._updater = new CTScheduledUpdater(this.update.bind(this), kUpdateFrequency);
 
         this.treeStatuses = {};
@@ -72,8 +73,9 @@ found in the LICENSE file.
       },
 
       update: function() {
+        window.console.log('unexpected-failures: update');

[ojan, 2014/08/20 18:47:51]

Ditto

[Mathieu, 2014/08/21 14:08:21]

Oops! Proof I'm human!

         // FIXME: These shouldn't update if there's already an update in progress.
-        this.revisionLog.update();
+        this.commitLog.update();
         this.$.analyzer.update();
         Object.keys(this.treeStatuses, function(tree, status) {
           status.update();