We should not allow navigation initiated from an onberforeunload handler.
<rdar://problem/31155736>
Created attachment 304947 [details] Patch and layout test
Comment on attachment 304947 [details] Patch and layout test View in context: https://bugs.webkit.org/attachment.cgi?id=304947&action=review > Source/WebCore/ChangeLog:3 > + onbeforeunload handler may start new navigation Would be nice to phrase this more actively: "onbeforeunload handler may start new navigation but should not be able to", or, better, "Prevent new navigations from onbeforeunload handler"
Comment on attachment 304947 [details] Patch and layout test View in context: https://bugs.webkit.org/attachment.cgi?id=304947&action=review > LayoutTests/fast/events/before-unload-forbidden-navigation.html:15 > +var didFireOnBeforeLoad = false; Will rename to didFireOnbeforeunloadEvent before landing. I am open to suggestions.
Created attachment 304948 [details] Patch and layout test Updated ChangeLog to reflect updated bug title. Also renamed the variable didFireOnBeforeLoad to didFireBeforeUnloadEvent in the included test.
Comment on attachment 304948 [details] Patch and layout test Clearing flags on attachment: 304948 Committed r214194: <http://trac.webkit.org/changeset/214194>
All reviewed patches have been landed. Closing bug.
*** Bug 169833 has been marked as a duplicate of this bug. ***
Mass move bugs into the DOM component.