Summary: | Tapping buttons in Data Detectors lookup previews doesn't work | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tim Horton <thorton> | ||||||||
Component: | New Bugs | Assignee: | Tim Horton <thorton> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | bdakin, commit-queue, dino, jbedard, megan_gardner, simon.fraser, webkit-bug-importer, wenson_hsieh | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Tim Horton
2019-08-09 10:17:19 PDT
Created attachment 375932 [details]
Patch
Created attachment 375933 [details]
Patch
Comment on attachment 375933 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=375933&action=review > Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:8298 > + if ([configuration isKindOfClass:getDDContextMenuConfigurationClass()]) { File a bug to clean this all up once the SPI has settled? > Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:8305 > + shouldExpandPreview = !!ddConfiguration.interactionViewControllerProvider; I find it odd that the SPI had a var for this, but now does not? But I'm assuming you know what's going on here.. Comment on attachment 375933 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=375933&action=review >> Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:8298 >> + if ([configuration isKindOfClass:getDDContextMenuConfigurationClass()]) { > > File a bug to clean this all up once the SPI has settled? Sure >> Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:8305 >> + shouldExpandPreview = !!ddConfiguration.interactionViewControllerProvider; > > I find it odd that the SPI had a var for this, but now does not? But I'm assuming you know what's going on here.. They upgraded from a bool to a nullable block that returns a VC 🤷♂️ Committed r248469: <https://trac.webkit.org/changeset/248469> This broke the iOS 13 seed build...we don't have bots for it yet, but I've been building it pretty regularly locally looking for this sort of thing. Reopening to attach new patch. Created attachment 376073 [details]
Patch
Comment on attachment 376073 [details] Patch Clearing flags on attachment: 376073 Committed r248535: <https://trac.webkit.org/changeset/248535> All reviewed patches have been landed. Closing bug. |