Summary: | Rename _highlightLongPressCanClick and only add gesture recognizer when necessary | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dean Jackson <dino> | ||||
Component: | New Bugs | Assignee: | Dean Jackson <dino> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | graouts, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Dean Jackson
2019-04-24 04:55:34 PDT
Created attachment 368119 [details]
Patch
Comment on attachment 368119 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=368119&action=review > Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:756 > + } Maybe I'm not used to this kind of use of a block, but this looks a bit nasty to have this code only be conditionalized with a build-time flag. Could we have this wrapped in a function with an early return if the build-time flag and condition are as expected? Committed r244586: <https://trac.webkit.org/changeset/244586> |