21699 2008-10-16 18:10:21 -0700 onbeforeunload/onunload should fire on frames/iframes 2009-07-16 02:40:08 -0700 1 1 1 Unclassified WebKit WebCore JavaScript 528+ (Nightly build) PC OS X 10.5 RESOLVED DUPLICATE 19418 P2 Normal --- 1 ojan webkit-unassigned ap bedney dpopa oliver ted oldest_to_newest 95679 0 ojan 2008-10-16 18:10:21 -0700 Testcase coming. 95682 1 24449 ojan 2008-10-16 18:33:11 -0700 Created attachment 24449 reduced testcase page with an iframe that has an onbeforeunload handler. attempting to leave the page should popup the beforeunload dialog. 98047 2 ted 2008-11-08 17:14:51 -0800 This looks like a duplicate of bug #19418. 98275 3 ap 2008-11-11 02:44:41 -0800 See also: bug 15652. 98857 4 ap 2008-11-15 02:22:37 -0800 Confirmed with r38387. In bug #19418, the handler is installed on the body of iframe document, while here, it is installed on Window object. So, these bugs are not exact duplicates, keeping them separate for now. 98858 5 ap 2008-11-15 02:29:51 -0800 Note however that fixing this may give a way to ads in iframes to be even more nasty than they are now, and open windows when navigating away from a page with an ad. They could be other undesirable consequences, as well. 119278 6 dpopa 2009-04-29 02:48:59 -0700 (In reply to comment #5) > Note however that fixing this may give a way to ads in iframes to be even more > nasty than they are now, and open windows when navigating away from a page with > an ad. They could be other undesirable consequences, as well. > From your argument I can see you are against people having guns, because, you say, some will start shooting other people. The beforeunload event could be abused in the same way as the unload event. People already have methods of killing other people. If the beforeunload event does not trigger for iframes, developers have no way of stopping a navigation inside an IFRAME so the user will not have a chance to save their work. I'm not saying having guns is a good thing, I'm just saying that the beforeunload is really useful. 126316 7 mjs 2009-06-16 14:58:42 -0700 I'm told (though I haven't tested) that IE and Firefox allow this. The HTML5 spec also describes beforeunload in some detail. 126318 8 bedney 2009-06-16 15:10:13 -0700 Maciej - Yes, I know for a fact that IE and FF allow this, because I use it all day, every day. For Webkit-based browsers, I had to play major workaround games to get this to work. Cheers, - Bill 132210 9 oliver 2009-07-16 02:40:08 -0700 *** This bug has been marked as a duplicate of bug 19418 *** 24449 2008-10-16 18:33:11 -0700 2008-10-16 18:33:11 -0700 reduced testcase iframe.html text/html 315 ojan PElGUkFNRSBvbmxvYWQ9InJlZ2lzdGVyVW5sb2FkSGFuZGxlcigpIj48L0lGUkFNRT4KVGhpcyBw YWdlIGRvZXMgbm90IGhhdmUgYW4gb25iZWZvcmV1bmxvYWQgaGFuZGxlci4KPHNjcmlwdD4KZnVu Y3Rpb24gcmVnaXN0ZXJVbmxvYWRIYW5kbGVyKCkgewogIHZhciBpZnIgPSBkb2N1bWVudC5nZXRF bGVtZW50c0J5VGFnTmFtZSgnaWZyYW1lJylbMF07CiAgaWZyLmNvbnRlbnRXaW5kb3cub25iZWZv cmV1bmxvYWQgPSBmdW5jdGlvbigpIHsKICAgIHJldHVybiAnWW91IHNob3VsZCBiZSBzZWVpbmcg YSBjaGVjayBub3cuJzsKICB9Cn0KPC9zY3JpcHQ+