Summary: | [iOS] beforeunload event does not fire in MobileSafari | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||||
Component: | DOM | Assignee: | Chris Dumez <cdumez> | ||||||||
Status: | REOPENED --- | ||||||||||
Severity: | Normal | CC: | achristensen, darin, ews-watchlist, ggaren, japhet, sam, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Chris Dumez
2020-11-18 11:35:14 PST
Created attachment 414464 [details]
Patch
Comment on attachment 414464 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=414464&action=review r=me > Source/WebCore/ChangeLog:9 > + MobileSafari on iOS does implement WKUIDelegate's runJavaScriptAlertPanelWithMessage because does not > Source/WebCore/ChangeLog:16 > + Might be worth mentioning that iOS already fires the unload event, so running an event at this time is not a significant change in policy. Created attachment 414466 [details]
Patch
Created attachment 414470 [details]
Patch
Committed r269983: <https://trac.webkit.org/changeset/269983> All reviewed patches have been landed. Closing bug and clearing flags on attachment 414470 [details]. Reverted r269983 for reason: Seems to have regressed PLT5 Committed r270560: <https://trac.webkit.org/changeset/270560> |