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

Side by Side Diff: LayoutTests/inspector-protocol/page/javascriptDialogEvents.html

Issue 158783002: Limit beforeunload returnValue propagation to event handlers. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Switch tests to use onbeforeunload event handlers instead. Created 6 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
OLDNEW
1 <html> 1 <html>
2 <head> 2 <head>
3 <script type="text/javascript" src="../../http/tests/inspector-protocol/resource s/protocol-test.js"></script> 3 <script type="text/javascript" src="../../http/tests/inspector-protocol/resource s/protocol-test.js"></script>
4 <script> 4 <script>
5 5
6 window.addEventListener("beforeunload", onBeforeUnload); 6 window.onbeforeunload = onBeforeUnload;
7 7
8 function onBeforeUnload() 8 function onBeforeUnload()
9 { 9 {
10 window.removeEventListener("beforeunload", onBeforeUnload); 10 window.removeEventListener("beforeunload", onBeforeUnload);
11 return "beforeunload in javascriptDialogEvents"; 11 return "beforeunload in javascriptDialogEvents";
12 } 12 }
13 13
14 function test() 14 function test()
15 { 15 {
16 InspectorTest.eventHandler["Page.javascriptDialogOpening"] = onOpening; 16 InspectorTest.eventHandler["Page.javascriptDialogOpening"] = onOpening;
(...skipping 26 matching lines...) Expand all
43 { 43 {
44 InspectorTest.completeTest(); 44 InspectorTest.completeTest();
45 } 45 }
46 } 46 }
47 47
48 </script> 48 </script>
49 </head> 49 </head>
50 <body onload="runTest()"> 50 <body onload="runTest()">
51 </body> 51 </body>
52 </html> 52 </html>
OLDNEW
« no previous file with comments | « LayoutTests/fast/loader/form-submission-after-beforeunload-cancel.html ('k') | Source/bindings/v8/V8AbstractEventListener.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698