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

tracing/tracing/model/thread.html

Index: tracing/tracing/model/thread.html
diff --git a/tracing/tracing/model/thread.html b/tracing/tracing/model/thread.html
index 16d8ea63a4131ce2247448b70d0eaa22b288bbd8..0bad04ee5e31521f19ff7da03e5c0908748d8d73 100644
--- a/tracing/tracing/model/thread.html
+++ b/tracing/tracing/model/thread.html
@@ -73,10 +73,6 @@ tr.exportTo('tr.model', function() {
       return Thread.compare(this, that);
     },
 
-    findTopmostSlicesInThisContainer: function(eventPredicate, callback,
-                                               opt_this) {
-    },
-
     childEventContainers: function*() {
       if (this.sliceGroup.length)
         yield this.sliceGroup;
@@ -86,11 +82,9 @@ tr.exportTo('tr.model', function() {
         yield this.asyncSliceGroup;
     },
 
-    childEvents: function*(eventTypePredicate, opt_this) {
-      if (this.timeSlices && this.timeSlices.length) {
-        if (eventTypePredicate.call(opt_this, ThreadTimeSlice))
-          yield * this.timeSlices;
-      }
+    childEvents: function*() {
+      if (this.timeSlices && this.timeSlices.length)

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

you don't need "&& this.timeSlices.length" here, right? Otherwise, yield *
should just yield an empty array, which is the same as if we hadn't yielded
anything

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

Done

+        yield * this.timeSlices;
     },
 
     iterateAllPersistableObjects: function(cb) {