[DevTools] Removed InspectorTest.invokeFunctionInPageAsync

third_party/WebKit/LayoutTests/http/tests/inspector/timeline-test.js

 function wrapCallFunctionForTimeline(f)
 {
     var script = document.createElement("script");
     script.textContent = "(" + f.toString() + ")()\n//# sourceURL=wrapCallFunctionForTimeline.js";
     document.body.appendChild(script);
 }
 
 var initialize_Timeline = function() {
 
 InspectorTest.preloadPanel("timeline");
@@ skipping 75 matching lines @@
     var categories = "-*,disabled-by-default-devtools.timeline*,devtools.timeline," + WebInspector.TracingModel.TopLevelEventCategory;
     if (additionalCategories)
         categories += "," + additionalCategories;
     var timelinePanel = WebInspector.panels.timeline;
     var timelineController = InspectorTest.timelineController();
     timelinePanel._timelineController = timelineController;
     timelineController._startRecordingWithCategories(categories, enableJSSampling, tracingStarted);
 
     function tracingStarted()
     {
-        InspectorTest.invokePageFunctionAsync(functionName, onPageActionsDone);
+        InspectorTest.callFunctionInPageAsync(functionName).then(onPageActionsDone);
     }
 
     function onPageActionsDone()
     {
         InspectorTest.addSniffer(WebInspector.panels.timeline, "loadingComplete", callback)
         timelineController.stopRecording();
     }
 }
 
 InspectorTest.timelineModel = function()
@@ skipping 59 matching lines @@
     {
         InspectorTest.stopTimeline(doneCallback);
     }
 }
 
 InspectorTest.invokeAsyncWithTimeline = function(functionName, doneCallback)
 {
     InspectorTest.startTimeline(step1);
     function step1()
     {
-        InspectorTest.invokePageFunctionAsync(functionName, step2);
+        InspectorTest.callFunctionInPageAsync(functionName).then(step2);
     }
 
     function step2()
     {
         InspectorTest.stopTimeline(InspectorTest.safeWrap(doneCallback));
     }
 }
 
 InspectorTest.loadTimelineRecords = function(records)
 {
@@ skipping 267 matching lines @@
     {
         return new InspectorTest.FakeFileReader(timelineData, delegate, timeline._saveToFile.bind(timeline));
     }
 
     InspectorTest.override(WebInspector.TimelineLoader, "_createFileReader", createFileReader);
     timeline._loadFromFile({});
 }
 
 };
 
-function generateFrames(count, callback)
+function generateFrames(count)
 {
-    makeFrame();
-    function makeFrame()
-    {
-        document.body.style.backgroundColor = count & 1 ? "rgb(200, 200, 200)" : "rgb(240, 240, 240)";
-        if (!--count) {
-            callback();
-            return;
-        }
-        if (window.testRunner)
-            testRunner.capturePixelsAsyncThen(requestAnimationFrame.bind(window, makeFrame));
-        else
-            window.requestAnimationFrame(makeFrame);
+    var promise = Promise.resolve();
+    for (let i = count; i > 0; --i) {
+        promise = promise
+            .then(() => document.body.style.backgroundColor = i & 1 ? "rgb(200, 200, 200)" : "rgb(240, 240, 240)")
+            .then(() => waitForFrame());

[caseq, 2016/08/10 22:44:55]

merge into above?

[kozy, 2016/08/11 01:34:16]

Done.

     }
+    return promise;
 }
+
+function waitForFrame()
+{
+    var callback;
+    var promise = new Promise((fulfill) => callback = fulfill);
+    if (window.testRunner)

[caseq, 2016/08/10 22:44:55]

inline into the Promise() constructor?

[kozy, 2016/08/11 01:34:16]

Acknowledged.

+        testRunner.capturePixelsAsyncThen(() => window.requestAnimationFrame(callback));
+    else
+        window.requestAnimationFrame(callback);
+    return promise;
+}