Bug 200557 - REGRESSION (52279987): Most of the WKUIDelegate contextMenu delegate methods are not being called
Summary: REGRESSION (52279987): Most of the WKUIDelegate contextMenu delegate methods ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dean Jackson
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-08-08 18:13 PDT by Dean Jackson
Modified: 2019-08-09 08:17 PDT (History)
2 users (show)

See Also:


Attachments
Patch (3.62 KB, patch)
2019-08-08 18:17 PDT, Dean Jackson
wenson_hsieh: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dean Jackson 2019-08-08 18:13:28 PDT
REGRESSION (52279987): Most of the WKUIDelegate contextMenu delegate methods are not being called
Comment 1 Dean Jackson 2019-08-08 18:13:54 PDT
<rdar://problem/54099201>
Comment 2 Dean Jackson 2019-08-08 18:17:00 PDT
Created attachment 375876 [details]
Patch
Comment 3 Dean Jackson 2019-08-08 18:51:21 PDT
Actually <rdar://problem/53717962> now.
Comment 4 Dean Jackson 2019-08-08 19:11:52 PDT
Committed r248456: <https://trac.webkit.org/changeset/248456>
Comment 5 Darin Adler 2019-08-09 08:07:28 PDT
Does this break compatible with the one-version-back iOS? Don’t we normally keep that working for test purposes?
Comment 6 Wenson Hsieh 2019-08-09 08:08:54 PDT
(In reply to Darin Adler from comment #5)
> Does this break compatible with the one-version-back iOS? Don’t we normally
> keep that working for test purposes?

UIContextMenuInteraction is new in iOS 13, so this should only break compatibility with older versions of iOS 13.
Comment 7 Darin Adler 2019-08-09 08:17:00 PDT
(In reply to Wenson Hsieh from comment #6)
> (In reply to Darin Adler from comment #5)
> > Does this break compatible with the one-version-back iOS? Don’t we normally
> > keep that working for test purposes?
> 
> UIContextMenuInteraction is new in iOS 13, so this should only break
> compatibility with older versions of iOS 13.

OK, great.