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

Side by Side Diff: LayoutTests/fast/workers/worker-error-handler-workerglobalscope.html

Issue 607513002: Onerror return value handling in workers fixed as per HTML5 spec. Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Added new layout tests and rebaselined two of them. Created 6 years, 2 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 unified diff | Download patch
OLDNEW
(Empty)
1 <!DOCTYPE html>
2 <html>
3 <body>
4 <p>Testing worker error handler invocation order.</p>
5 <div id="result"></div>
6 <script type="text/javascript">
7 var windowErrorHandlerInvoked = false;
8 var workerErrorHandlerInvoked = false;
9 var workerGlobalScopeErrorHandlerInvoked = false;
10 function log(message)
11 {
12 document.getElementById("result").innerHTML += message + "</br>";
13 }
14
15 if (window.testRunner) {
16 testRunner.dumpAsText();
17 testRunner.waitUntilDone();
18 }
19
20 window.onerror = function (e) {
21 windowErrorHandlerInvoked = true;
22 log("FAIL: Window error handler should not be invoked!");
23 }
24
25 var worker = new Worker('resources/worker-error-handler-workerglobalscope.js');
26 worker.postMessage("Start");
27
28 worker.onmessage = function(evt) {
29 if (evt.data == "DONE") {
30 if(!windowErrorHandlerInvoked && !workerErrorHandlerInvoked) {
31 log("TEST PASS.");
32 }
33 else {
34 log("TEST FAIL.")
35 }
36 log("DONE.")
37 if (window.testRunner)
38 testRunner.notifyDone();
39 }
40 else if (evt.data == "workerGlobalScopeErrorHandlerInvoked") {
41 workerGlobalScopeErrorHandlerInvoked = true;
42 log("PASS: WorkerGlobalScope error handler invoked.");
43 }
44 }
45 worker.onerror = function (e) {
46 workerErrorHandlerInvoked = true;
47 log("FAIL: Worker error handler should not be invoked!");
48 return false;
49 }
50 </script>
51 </body>
52 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698