Summary: | pointerevents/ios/touch-action-none-in-overflow-scrolling-touch.html is a timeout | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Antoine Quint <graouts> | ||||
Component: | New Bugs | Assignee: | Antoine Quint <graouts> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | koivisto, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Antoine Quint
2019-05-09 06:59:17 PDT
Created attachment 369490 [details]
Patch
Comment on attachment 369490 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=369490&action=review > Source/WebKit/UIProcess/RemoteLayerTree/ios/ScrollingTreeScrollingNodeDelegateIOS.mm:345 > + m_activeTouchActions = scrollingCoordinatorProxy.activeTouchActionsForTouchIdentifier(*touchIdentifier); Should this be cleared when the scrolling fully finishes? Seem bet weird that it can stay non-empty long past the gesture. (In reply to Antti Koivisto from comment #3) > Comment on attachment 369490 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=369490&action=review > > > Source/WebKit/UIProcess/RemoteLayerTree/ios/ScrollingTreeScrollingNodeDelegateIOS.mm:345 > > + m_activeTouchActions = scrollingCoordinatorProxy.activeTouchActionsForTouchIdentifier(*touchIdentifier); > > Should this be cleared when the scrolling fully finishes? Seem bet weird > that it can stay non-empty long past the gesture. Fixing this in commit. Committed r245134: <https://trac.webkit.org/changeset/245134> |