Bug 215988 - REGRESSION (r266262): Scroll latching fails when scrolling into a preventDefault() region
Summary: REGRESSION (r266262): Scroll latching fails when scrolling into a preventDefa...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Scrolling (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Simon Fraser (smfr)
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-08-30 11:10 PDT by Simon Fraser (smfr)
Modified: 2020-08-30 15:09 PDT (History)
9 users (show)

See Also:

Attachments
Testcase (1010 bytes, text/html)
2020-08-30 11:10 PDT, Simon Fraser (smfr) 		no flags Details
Patch (6.46 KB, patch)
2020-08-30 13:48 PDT, Simon Fraser (smfr) 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Fraser (smfr) 2020-08-30 11:10:09 PDT 
Created attachment 407570 [details]
Testcase

In the attached testcase, put your pointer where indicated and momentum scroll down.

Scrolling stops as soon as the pointer is over the wheel event handler region, but it should not.
Comment 1 Radar WebKit Bug Importer 2020-08-30 11:10:35 PDT 
<rdar://problem/68034716>
Comment 2 Simon Fraser (smfr) 2020-08-30 12:35:11 PDT 
This happens because m_lastLatchedNodeInterationTime isn't updated, so we lose latching 100ms after the start of the gesture.
Comment 3 Simon Fraser (smfr) 2020-08-30 13:48:13 PDT 
Created attachment 407572 [details]
Patch
Comment 4 EWS 2020-08-30 15:09:26 PDT 
Committed r266336: <https://trac.webkit.org/changeset/266336>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 407572 [details].