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

Side by Side Diff: LayoutTests/fast/workers/resources/worker-thread-multi-port.js

Issue 1025203002: Have postMessage()s throw TypeError on failed arity checks. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: tidy up incorrect postMessage() usage Created 5 years, 9 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
1 onmessage = function(event) { 1 onmessage = function(event) {
2 if (event.data == "noport") { 2 if (event.data == "noport") {
3 if (event.ports && !event.ports.length) 3 if (event.ports && !event.ports.length)
4 testPassed("event.ports is non-null and zero length when no port sen t"); 4 testPassed("event.ports is non-null and zero length when no port sen t");
5 else 5 else
6 testFailed("event.ports is null or non-zero length when no port sent "); 6 testFailed("event.ports is null or non-zero length when no port sent ");
7 } else if (event.data == "zero ports") { 7 } else if (event.data == "zero ports") {
8 if (event.ports && !event.ports.length) 8 if (event.ports && !event.ports.length)
9 testPassed("event.ports is non-null and zero length when empty array sent"); 9 testPassed("event.ports is non-null and zero length when empty array sent");
10 else 10 else
11 testFailed("event.ports is null or non-zero length when empty array sent"); 11 testFailed("event.ports is null or non-zero length when empty array sent");
12 } else if (event.data == "two ports") { 12 } else if (event.data == "two ports") {
13 if (!event.ports) { 13 if (!event.ports) {
14 testFailed("event.ports should be non-null when ports sent"); 14 testFailed("event.ports should be non-null when ports sent");
15 return; 15 return;
16 } 16 }
17 if (event.ports.length == 2) 17 if (event.ports.length == 2)
18 testPassed("event.ports contains two ports when two ports sent"); 18 testPassed("event.ports contains two ports when two ports sent");
19 else 19 else
20 testFailed("event.ports contained " + event.ports.length + " when tw o ports sent"); 20 testFailed("event.ports contained " + event.ports.length + " when tw o ports sent");
21 } else if (event.data == "failed ports") { 21 } else if (event.data == "failed ports") {
22 if (event.ports.length == 2) 22 if (event.ports.length == 2)
23 testPassed("event.ports contains two ports when two ports re-sent af ter error"); 23 testPassed("event.ports contains two ports when two ports re-sent af ter error");
24 else 24 else
25 testFailed("event.ports contained " + event.ports.length + " when tw o ports re-sent after error"); 25 testFailed("event.ports contained " + event.ports.length + " when tw o ports re-sent after error");
26 } else if (event.data == "done") { 26 } else if (event.data == "done") {
27 postMessage("done"); 27 postMessage("done");
28 event.ports[0].postMessage("done"); 28 event.ports[0].postMessage("done");
29 } else if (event.data == "noargs") {
30 try {
31 postMessage();
32 testFailed("postMessage() did not throw");
33 } catch (e) {
34 testPassed("postMessage() threw exception: " + e);
35 }
29 } else 36 } else
30 testFailed("Received unexpected message: " + event.data); 37 testFailed("Received unexpected message: " + event.data);
31 }; 38 };
32 39
33 function testPassed(msg) 40 function testPassed(msg)
34 { 41 {
35 postMessage("PASS"+msg); 42 postMessage("PASS"+msg);
36 } 43 }
37 44
38 function testFailed(msg) 45 function testFailed(msg)
39 { 46 {
40 postMessage("FAIL"+msg); 47 postMessage("FAIL"+msg);
41 } 48 }
OLDNEW
« no previous file with comments | « LayoutTests/fast/workers/resources/worker-multi-port.js ('k') | LayoutTests/fast/workers/worker-multi-port-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698