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

tracing/tracing/model/power_series_test.html

Index: tracing/tracing/model/power_series_test.html
diff --git a/tracing/tracing/model/power_series_test.html b/tracing/tracing/model/power_series_test.html
index 331307d7e26d277678c5d88ca2940ef173762538..3f0a51bbfcd0acbd46065f2cb0e736584b1c05b3 100644
--- a/tracing/tracing/model/power_series_test.html
+++ b/tracing/tracing/model/power_series_test.html
@@ -128,30 +128,5 @@ tr.b.unittest.testSuite(function() {
     assert.equal(series.bounds.min, 0);
     assert.equal(series.bounds.max, 4);
   });
-

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

What's the reason for deleting this? It seems like it's testing that all of the
right stuff is being added in this.childEvents()

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

Put test back in

-  test('iterateAllEventsInThisContainer', function() {
-    var series = new PowerSeries(new Model().device);
-
-    var eventsInSeries = [];
-    series.iterateAllEventsInThisContainer(
-        function() { return true; },
-        function(event) {
-          eventsInSeries.push(event);
-        });
-
-    assert.deepEqual(eventsInSeries, []);
-
-    var sample1 = series.addPowerSample(0, 1);
-    var sample2 = series.addPowerSample(1, 2);
-
-    eventsInSeries = [];
-    series.iterateAllEventsInThisContainer(
-        function() { return true; },
-        function(event) {
-          eventsInSeries.push(event);
-        });
-
-    assert.deepEqual(eventsInSeries, [sample1, sample2]);
-  });
 });
 </script>