Chromium Code Reviews| Index: third_party/WebKit/LayoutTests/inspector-protocol/worker/worker-console.html |
| diff --git a/third_party/WebKit/LayoutTests/inspector-protocol/worker/worker-console.html b/third_party/WebKit/LayoutTests/inspector-protocol/worker/worker-console.html |
| index bc201b9941ee147f29cbf0b3fa7c49beed4db5c9..676145ad9b63f766062047b130ed652e5a0da78e 100644 |
| --- a/third_party/WebKit/LayoutTests/inspector-protocol/worker/worker-console.html |
| +++ b/third_party/WebKit/LayoutTests/inspector-protocol/worker/worker-console.html |
| @@ -39,10 +39,9 @@ function stopWorker() |
| function test() |
| { |
| var workerEventHandler = []; |
|
kozy
2016/07/12 00:45:19
It should be an object.
dgozman
2016/07/12 01:16:38
Done.
|
| - InspectorTest.eventHandler["Console.messageAdded"] = onConsoleMessage.bind(null, false); |
| InspectorTest.eventHandler["Worker.workerCreated"] = onWorkerCreated; |
| InspectorTest.eventHandler["Worker.dispatchMessageFromWorker"] = onWorkerMessage; |
| - workerEventHandler["Console.messageAdded"] = onConsoleMessage.bind(null, true); |
| + workerEventHandler["Console.messageAdded"] = onConsoleMessageFromWorker; |
| var workerId; |
| @@ -91,20 +90,25 @@ function test() |
| function logInWorker(message, next) |
| { |
| InspectorTest.log("Logging in worker: " + message); |
| - if (next) |
| - InspectorTest.evaluateInPageAsync("logInWorkerFromPage(\"" + message + "\", %callback)", next); |
| - else |
| - InspectorTest.evaluateInPage("logInWorkerFromPage(\"" + message + "\")"); |
| + InspectorTest.eventHandler["Console.messageAdded"] = onConsoleMessage; |
| + InspectorTest.evaluateInPage("logInWorkerFromPage(\"" + message + "\")"); |
| + |
| + function onConsoleMessage(payload) |
| + { |
| + InspectorTest.log("Got console message from page: " + payload.params.message.text); |
| + delete InspectorTest.eventHandler["Console.messageAdded"]; |
| + next(); |
| + } |
| } |
| var gotMessages = []; |
| var waitingForMessage; |
| var waitingForMessageCallback; |
| - function onConsoleMessage(fromWorker, payload) |
| + function onConsoleMessageFromWorker(payload) |
| { |
| var message = payload.params.message.text; |
| - InspectorTest.log("Got console message from " + (fromWorker ? "worker" : "page") + ": " + message); |
| + InspectorTest.log("Got console message from worker: " + message); |
| gotMessages.push(message); |
| if (message === waitingForMessage) |
| waitingForMessageCallback(); |
| @@ -140,8 +144,7 @@ function test() |
| function stop0(next) |
| { |
| InspectorTest.log("Stopping worker"); |
| - waitForMessage("message0", next); |
| - InspectorTest.evaluateInPage("stopWorker()"); |
| + InspectorTest.evaluateInPage("stopWorker()", next); |
| }, |
| function start1(next) |
| @@ -170,8 +173,17 @@ function test() |
| function log2(next) |
| { |
| + logInWorker("message2", next); |
| + }, |
| + |
| + function waitForMessage2(next) |
| + { |
| waitForMessage("message2", next); |
| - logInWorker("message2"); |
| + }, |
| + |
| + function throw1(next) |
| + { |
| + logInWorker("throw1", next); |
| }, |
| function disable1(next) |
| @@ -218,8 +230,12 @@ function test() |
| function log5(next) |
| { |
| + logInWorker("message5", next); |
| + }, |
| + |
| + function waitForMessage5(next) |
| + { |
| waitForMessage("message5", next); |
| - logInWorker("message5"); |
| }, |
| function stop2(next) |
| @@ -242,8 +258,7 @@ function test() |
| function stop3(next) |
| { |
| InspectorTest.log("Stopping worker"); |
| - waitForMessage("message6", next); |
| - InspectorTest.evaluateInPage("stopWorker()"); |
| + InspectorTest.evaluateInPage("stopWorker()", next); |
| }, |
| function disable2(next) |