Change call-sites in trace viewer to use generators instead of iteration functions.

tracing/tracing/model/process_base.html

Index: tracing/tracing/model/process_base.html
diff --git a/tracing/tracing/model/process_base.html b/tracing/tracing/model/process_base.html
index 16b67424b2c81a890ad3cc0213511883deb3d2f7..a44a0e2de4f44f20611463bc6179f18ff61160be 100644
--- a/tracing/tracing/model/process_base.html
+++ b/tracing/tracing/model/process_base.html
@@ -52,17 +52,14 @@ tr.exportTo('tr.model', function() {
       throw new Error('Not implemented');
     },
 
-    findTopmostSlicesInThisContainer: function(eventPredicate, callback,

[charliea (OOO until 10-5), 2016/06/29 18:08:45]

nit: what's the reason for deleting this in this CL?

[alexandermont, 2016/06/29 20:47:12]

Blank function is now default

-                                               opt_this) {
-    },
-
     childEventContainers: function*() {
       yield * tr.b.dictionaryValues(this.threads);
       yield * tr.b.dictionaryValues(this.counters);
       yield this.objects;
     },
 
-    childEvents: function*(eventTypePredicate, opt_this) {
+
+    childEvents: function*() {

[charliea (OOO until 10-5), 2016/06/29 18:08:45]

same note: isn't this just the default impl?

[alexandermont, 2016/06/29 20:47:12]

Removed

     },
 
     iterateAllPersistableObjects: function(cb) {
@@ -87,9 +84,8 @@ tr.exportTo('tr.model', function() {
      * specified.
      */
     shiftTimestampsForward: function(amount) {
-      this.iterateAllChildEventContainers(function(child) {
+      for (var child of this.childEventContainers())
         child.shiftTimestampsForward(amount);
-      });
     },
 
     /**