Summary: | [iOS] REGRESSION(r265088): "pointerdown" doesn't fire using a trackpad after double-tapping | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> | ||||||||
Component: | UI Events | Assignee: | Devin Rousso <hi> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | graouts, hi, thorton, webkit-bug-importer, wenson_hsieh | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=220074 | ||||||||||
Bug Depends on: | 214968 | ||||||||||
Bug Blocks: | 225734 | ||||||||||
Attachments: |
|
Description
Devin Rousso
2020-12-21 15:07:28 PST
Created attachment 416636 [details]
Patch
My understanding of UIKit gesture recognizers (and how we use them) and the purpose behind `m_activePointerIdsToCapturingData` is possibly lacking, so this may not be the best approach, but it seems logical that we would remove entries for touch interactions from `m_activePointerIdsToCapturingData` once the interaction finishes (e.g. `"touchend"`).
Created attachment 416638 [details] [HTML] LayoutTest to add after bug 220074 is resolved I'd like to add a test for this, but after talking with @Tim Horton we do not have a way to generate mouse interactions on iOS from within a LayoutTest. I've created bug 220074 to hopefully add this in the next few days :) In the meantime, here's the test that I would've added. Committed r271216: <https://trac.webkit.org/changeset/271216> All reviewed patches have been landed. Closing bug and clearing flags on attachment 416636 [details]. |