RESOLVED FIXED 150382
Single tapping clickable elements in unscalable pages should show a tap highlight 
https://bugs.webkit.org/show_bug.cgi?id=150382
Summary Single tapping clickable elements in unscalable pages should show a tap highl...
Wenson Hsieh
Reported 2015-10-20 18:48:07 PDT
Single tapping clickable elements in unscalable pages should show a tap highlight. Currently, tap highlights are broken for fast clicking in pages that are unscalable because WKContentView assumes that the tap highlight geometry will arrive from the web process before the single tap is committed by the gesture recognizer. However, when the double tap gesture recognizer is disabled ahead of time (as in the case of unscalable pages) the tap is committed before the geometries come in.
Attachments
Patch (4.11 KB, patch)
2015-10-21 14:14 PDT, Wenson Hsieh 		simon.fraser: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Wenson Hsieh
Comment 1 2015-10-20 18:50:02 PDT
<rdar://problem/23172720>
Wenson Hsieh
Comment 2 2015-10-21 14:14:32 PDT
Created attachment 263733 [details] Patch
Simon Fraser (smfr)
Comment 3 2015-10-21 14:58:12 PDT
Comment on attachment 263733 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=263733&action=review > Source/WebKit2/UIProcess/ios/WKContentViewInteraction.mm:1254 > + _isExpectingFastSingleTapCommit = ![_doubleTapGestureRecognizer isEnabled]; dot syntax?
Wenson Hsieh
Comment 4 2015-10-21 15:29:19 PDT
Comment on attachment 263733 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=263733&action=review >> Source/WebKit2/UIProcess/ios/WKContentViewInteraction.mm:1254 >> + _isExpectingFastSingleTapCommit = ![_doubleTapGestureRecognizer isEnabled]; > > dot syntax? Fixed!
Wenson Hsieh
Comment 5 2015-10-21 15:31:39 PDT
Committed r191409: <http://trac.webkit.org/changeset/191409>
Note You need to log in before you can comment on or make changes to this bug.