Bug 222111

Summary: Clean up ScrollableArea and ScrollAnimator API and convert less between offsets and positions
Product: WebKit Reporter: Martin Robinson <mrobinson>
Component: ScrollingAssignee: Martin Robinson <mrobinson>
Status: RESOLVED FIXED    
Severity: Normal CC: changseok, esprehn+autocc, ews-watchlist, fred.wang, glenn, kondapallykalyan, pdr, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 146696    
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Description Martin Robinson 2021-02-18 08:59:06 PST
The ScrollableArea and ScrollAnimator API is a bit confusing when it comes to scrolling to offsets and positions. It isn't always clear when these APIs will trigger an animation and we are missing some opportunities to use "native" animations on Mac. These APIs also require a lot of conversion back and forth between offsets and positions. Finally, cleaning up this API will allow us to use native animations to do scroll snapping during thumb dragging.
Comment 1 Martin Robinson 2021-02-19 05:01:20 PST
Created attachment 420957 [details]
Patch
Comment 2 Martin Robinson 2021-02-19 08:16:50 PST
Created attachment 420970 [details]
Patch
Comment 3 Martin Robinson 2021-02-22 00:28:25 PST
Created attachment 421167 [details]
Patch
Comment 4 EWS 2021-02-22 13:39:45 PST
Committed r273275: <https://commits.webkit.org/r273275>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 421167 [details].
Comment 5 Radar WebKit Bug Importer 2021-02-22 13:40:15 PST
<rdar://problem/74611765>