Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(160)

Unified Diff: third_party/WebKit/LayoutTests/inspector/tracing/worker-js-frames.html

Issue 2208963002: [DevTools] Removed InspectorTest.invokeFunctionInPageAsync (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove-promise-from-other-tests
Patch Set: a Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/inspector/tracing/worker-js-frames.html
diff --git a/third_party/WebKit/LayoutTests/inspector/tracing/worker-js-frames.html b/third_party/WebKit/LayoutTests/inspector/tracing/worker-js-frames.html
index 8be42e037fe0f628e2faf0ffc3ee2ac2061fcd36..efb2a9789b438363e56041c18d20ece08ec73196 100644
--- a/third_party/WebKit/LayoutTests/inspector/tracing/worker-js-frames.html
+++ b/third_party/WebKit/LayoutTests/inspector/tracing/worker-js-frames.html
@@ -33,8 +33,10 @@ function startWorkerAndRunTest()
}
}
-function startSecondWorker(onActionComplete)
+function startSecondWorker()
{
+ var callback;
+ var promise = new Promise((fulfill) => callback = fulfill);
// Make sure there is at least one trace event recorded for the first worker.
worker1.onmessage = gotResponseFromFirstWorker;
worker1.postMessage("");
@@ -50,9 +52,10 @@ function startSecondWorker(onActionComplete)
{
worker2.onmessage = null;
makeDOMEvent();
- onActionComplete();
+ callback();
}
}
+ return promise;
}
function test()

Powered by Google App Engine
This is Rietveld 408576698