third_party/WebKit/LayoutTests/http/tests/security/contentSecurityPolicy/connect-src-websocket-blocked.html - Issue 2456013002: CSP: 'connect-src' should not cause exceptions.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: third_party/WebKit/LayoutTests/http/tests/security/contentSecurityPolicy/connect-src-websocket-blocked.html

Issue 2456013002: CSP: 'connect-src' should not cause exceptions. (Closed)

Patch Set: Created 4 years, 1 month 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

« third_party/WebKit/LayoutTests/http/tests/security/contentSecurityPolicy/connect-src-eventsource-blocked.html ('K') | « third_party/WebKit/LayoutTests/http/tests/security/contentSecurityPolicy/connect-src-eventsource-redirect-to-blocked-expected.txt ('k') | third_party/WebKit/LayoutTests/http/tests/security/contentSecurityPolicy/connect-src-websocket-blocked-expected.txt » ('j') | third_party/WebKit/Source/modules/websockets/DOMWebSocket.cpp » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 <!DOCTYPE html>	1 <!DOCTYPE html>

2 <html>	2 <meta http-equiv="Content-Security-Policy" content="connect-src 'self'">

3 <head>	3 <script src="/resources/testharness.js"></script>

4 <meta http-equiv="Content-Security-Policy" content="connect-src ws://127.0.0.1:8 880">	4 <script src="/resources/testharnessreport.js"></script>

5 <script>	5 <script>

6 if (window.testRunner)	6 async_test(t => {

7 testRunner.dumpAsText();	7 var eventsSeen = 0;

	8

	9 var ws = new WebSocket("ws://example.test:8880/echo");

	10 ws.onopen = t.unreached_func("Open should not fire.");
	foolip 2016/10/27 13:59:50 Maybe lowercase open Maybe lowercase open Mike West 2016/10/27 14:18:24 Sure! Show quoted text On 2016/10/27 at 13:59:50, foolip wrote: > Maybe lowercase open Sure!
	11 ws.onerror = t.step_func(e => {

	12 assert_equals(ws.readyState, WebSocket.CLOSED);

	13

	14 eventsSeen++;

	15 if (eventsSeen == 2)

	16 t.done();

	17 });

	18

	19 document.addEventListener("securitypolicyviolation", t.step_func(e => {

	20 if (e.blockedURI != "ws://example.test:8880")

	21 return;

	22

	23 assert_equals(ws.readyState, WebSocket.CLOSED);

	24 assert_equals(e.violatedDirective, "connect-src");

	25

	26 eventsSeen++;

	27 if (eventsSeen == 2)

	28 t.done();

	29 }));

	30 }, "WebSocket should fire onerror.");
	foolip 2016/10/27 13:59:50 Maybe "error event" Maybe "error event" Mike West 2016/10/27 14:18:24 Sure! Show quoted text On 2016/10/27 at 13:59:50, foolip wrote: > Maybe "error event" Sure!
8 </script>	31 </script>

9 </head>

10 <body>

11 <pre id="console"></pre>

12 <script>

13 function log(msg)

14 {

15 document.getElementById("console").appendChild(document.createTextNode(msg + "\n"));

16 }

17

18 try {

19 var ws = new WebSocket("ws://localhost:8880/echo");

20 log("Fail");

21 } catch(e) {

22 log("Pass");

23 }

24

25 </script>

26 </body>

27 </html>

OLD	NEW