third_party/WebKit/LayoutTests/http/tests/security/referrer-policy-attribute-img-no-referrer-when-downgrade.html - Issue 1907413003: Wait until the test page is fully loaded before dumping the contents.

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/referrer-policy-attribute-img-no-referrer-when-downgrade.html

Issue 1907413003: Wait until the test page is fully loaded before dumping the contents. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Created 4 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

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

2 <html>	2 <html>

3 <head>	3 <head>

4 <script src="/resources/get-host-info.js"></script>	4 <script src="/resources/get-host-info.js"></script>

5 </head>	5 </head>

6 <body>	6 <body>

7 <script>	7 <script>

8 if (window.testRunner)	8 if (window.testRunner)

9 testRunner.waitUntilDone();	9 testRunner.waitUntilDone();

10	10

11 if (window.location.origin != get_host_info().HTTPS_ORIGIN) {	11 if (window.location.origin != get_host_info().HTTPS_ORIGIN) {

12 window.location = get_host_info().HTTPS_ORIGIN + window.location.pat hname;	12 window.location = get_host_info().HTTPS_ORIGIN + window.location.pat hname;

13 } else {	13 } else {

14 var img = document.createElement("img");	14 var img = document.createElement("img");

15 img.src = "http://127.0.0.1:8000/security/resources/green-if-no-refe rrer.php";	15 img.src = "http://127.0.0.1:8000/security/resources/green-if-no-refe rrer.php";

16 img.referrerpolicy = "no-referrer-when-downgrade";	16 img.referrerpolicy = "no-referrer-when-downgrade";

17 document.body.appendChild(img);	17 document.body.appendChild(img);

18 testRunner.notifyDone();	18

	19 // Before calling testRunner.notifyDone (and taking a screenshot of

	20 // actual.png), we need to make sure the image appended above got a

	21 // chance to paint itself.

	22 testRunner.layoutAndPaintAsyncThen(function() {
	dcheng 2016/04/25 23:15:03 Should this just use the image's load event? Should this just use the image's load event? Łukasz Anforowicz 2016/04/25 23:33:50 Done. You're right, the layoutAndPaintAsyncThen i Show quoted text On 2016/04/25 23:15:03, dcheng wrote: > Should this just use the image's load event? Done. You're right, the layoutAndPaintAsyncThen is wrong - I should have used the load event.
	23 testRunner.notifyDone();

	24 });

19 }	25 }

20 </script>	26 </script>

21 </body>	27 </body>

22 </html	28 </html>

OLD	NEW

« no previous file with comments | « third_party/WebKit/LayoutTests/http/tests/navigation/resources/notify-done.html ('k') | no next file » | no next file with comments »