LayoutTests/fast/dom/Window/Location/set-location-after-close.html - Issue 915033002: BatteryManager.getBattery(): gracefully bail on context-detached use.

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: LayoutTests/fast/dom/Window/Location/set-location-after-close.html

Issue 915033002: BatteryManager.getBattery(): gracefully bail on context-detached use. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: tidying up test Created 5 years, 10 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

« no previous file with comments | « LayoutTests/fast/dom/Window/Location/resources/set-location-after-close-new-window.html ('k') | LayoutTests/fast/files/file-reader-detached-no-crash.html » ('j') | LayoutTests/http/tests/navigation/beacon-detached-no-crash.html » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 <!DOCTYPE HTML>	1 <!DOCTYPE HTML>

2 <script src="../../../../resources/js-test.js"></script>	2 <script src="../../../../resources/js-test.js"></script>

3 <script>	3 <script>

4 description("Updating window.location after the window has been closed, without crashing.");	4 description("Updating window.location after the window has been closed, without crashing.");

5	5

6 // This tests opens a new window, sets a timeout to close it, then	6 // This tests opens a new window, sets a timeout to close it, then

7 // sets a timeout to navigate it via window.location. We pass if we	7 // sets a timeout to navigate it via window.location. We pass if we

8 // don't crash when setting the location of the already closed window,	8 // don't crash when setting the location of the already closed window,

9 // or when shutting down just after.	9 // or when shutting down just after.

10	10

11 if (window.testRunner) {	11 if (window.testRunner) {

12 testRunner.dumpAsText();	12 testRunner.dumpAsText();

13 testRunner.setCanOpenWindows();	13 testRunner.setCanOpenWindows();

14 testRunner.waitUntilDone();	14 testRunner.waitUntilDone();

15 }	15 }

16	16

17 var w = window.open('resources/set-location-after-close-new-window.html');	17 var w = window.open('../../../../resources/window-postmessage-open-close.html');

18	18

19 window.addEventListener("message", processMessage, false);	19 window.addEventListener("message", processMessage, false);

20	20

21 function finish() {	21 function finish() {

22 // Clear out window and force a GC to finalize the Window	22 // Clear out window and force a GC to finalize the Window

23 // object and its wrapper. Verifies that the shutdown is	23 // object and its wrapper. Verifies that the shutdown is

24 // done in an orderly manner.	24 // done in an orderly manner.

25 w = null;	25 w = null;

26 gc();	26 gc();

27 if (window.testRunner)	27 if (window.testRunner)

28 testRunner.notifyDone();	28 testRunner.notifyDone();

29 }	29 }

30	30

31 function processMessage(event) {	31 function processMessage(event) {

32 if (event.data == "opened")	32 if (event.data == "opened")

33 w.close();	33 w.close();

34 else if (event.data == "closed") {	34 else if (event.data == "closed") {

35 w.location = '';	35 w.location = '';

36 setInterval(finish, 0);	36 setInterval(finish, 0);

37 }	37 }

38 }	38 }

39 </script>	39 </script>

OLD	NEW