RESOLVED FIXED 231395
Minor refactoring to use ScrollUpdate in more places 
https://bugs.webkit.org/show_bug.cgi?id=231395
Summary Minor refactoring to use ScrollUpdate in more places
Simon Fraser (smfr)
Reported 2021-10-07 15:08:45 PDT
Minor refactoring to use ScrollUpdate in more places
Attachments
Patch (14.36 KB, patch)
2021-10-07 15:11 PDT, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2021-10-07 15:11:09 PDT
Created attachment 440544 [details] Patch
Myles C. Maxfield
Comment 2 2021-10-07 15:17:53 PDT
Comment on attachment 440544 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=440544&action=review > Source/WebCore/page/scrolling/AsyncScrollingCoordinator.cpp:263 > + auto scrollUpdate = ScrollUpdate { scrollingNodeID, scrollPosition, { }, ScrollingLayerPositionAction::Set }; Why not ScrollUpdate scrollUpdate(scrollingNodeID, scrollPosition, { }, ScrollingLayerPositionAction::Set);? Or even applyScrollUpdate(ScrollUpdate(scrollingNodeID, scrollPosition, { }, ScrollingLayerPositionAction::Set), ScrollType::Programmatic);? > Source/WebCore/page/scrolling/AsyncScrollingCoordinator.cpp:309 > + applyScrollPositionUpdate(WTFMove(update), ScrollType::User); Cool. > Source/WebCore/page/scrolling/ScrollingCoordinatorTypes.h:105 > +struct ScrollUpdate { If you're going to move this, why not move it into its own file?
EWS
Comment 3 2021-10-07 17:32:39 PDT
Committed r283761 (242681@main): <https://commits.webkit.org/242681@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 440544 [details].
Radar WebKit Bug Importer
Comment 4 2021-10-07 17:33:18 PDT
<rdar://problem/84007408>
Note You need to log in before you can comment on or make changes to this bug.