third_party/WebKit/LayoutTests/external/wpt/content-security-policy/style-src/style-src-imported-style-blocked.html - Issue 2784543003: Fixed wpt content-security-policy/style-src tests

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/external/wpt/content-security-policy/style-src/style-src-imported-style-blocked.html

Issue 2784543003: Fixed wpt content-security-policy/style-src tests (Closed)

Patch Set: Created 3 years, 8 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

« third_party/WebKit/LayoutTests/external/wpt/content-security-policy/style-src/style-src-imported-style-allowed.html ('K') | « third_party/WebKit/LayoutTests/external/wpt/content-security-policy/style-src/style-src-imported-style-allowed.html ('k') | third_party/WebKit/LayoutTests/external/wpt/content-security-policy/style-src/style-src-injected-inline-style-allowed.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 <!doctype html>

	2 <html>

	3 <head>

	4 <meta http-equiv="Content-Security-Policy" content="style-src 'self';">

	5 <script src="/resources/testharness.js"></script>

	6 <script src="/resources/testharnessreport.js"></script>

	7

	8 <script>

	9 var t = async_test("@import stylesheet should not load because it does not match style-src");

	10 var t_spv = async_test("Should fire a securitypolicyviolation event");

	11

	12 document.addEventListener("securitypolicyviolation", t_spv.step_func_done( function(e) {

	13 assert_equals("style-src", e.violatedDirective);

	14 }));

	15 </script>

	16 <link href="/content-security-policy/style-src/resources/style-src-import.cs s?pipe=sub" rel=stylesheet type=text/css>
	Mike West 2017/03/29 08:38:01 Ah, I see. You're using `?pipe=sub` instead of ren Ah, I see. You're using `?pipe=sub` instead of renaming the file. Was that a conscious decision, or just copy/paste? If the latter, I'd prefer to see it changed, as it forces the callsites to remember to do some work that's better avoided. andypaicu 2017/03/29 09:55:32 Just a copy/paste. Removed the pipe=sub Show quoted text On 2017/03/29 at 08:38:01, Mike West wrote: > Ah, I see. You're using `?pipe=sub` instead of renaming the file. Was that a conscious decision, or just copy/paste? If the latter, I'd prefer to see it changed, as it forces the callsites to remember to do some work that's better avoided. Just a copy/paste. Removed the pipe=sub
	17 </head>

	18 <body>

	19 <div id='log'></div>

	20

	21 <div id="content">Lorem ipsum</div>

	22

	23 <script>

	24 t.step(function() {

	25 var contentEl = document.getElementById("content");

	26 var marginLeftVal = getComputedStyle(contentEl).getPropertyValue('margin -left');

	27 assert_false(marginLeftVal == "2px");

	28 t.done();

	29 });

	30 </script>

	31

	32 </body>

	33 </html>

OLD	NEW