RESOLVED FIXED 231238
Share macOS code between ScrollAnimator::handleWheelEvent() and ScrollingEffectsController::handleWheelEvent() 
https://bugs.webkit.org/show_bug.cgi?id=231238
Summary Share macOS code between ScrollAnimator::handleWheelEvent() and ScrollingEffe...
Simon Fraser (smfr)
Reported 2021-10-05 10:14:36 PDT
There's duplicate but slightly different code here and it needs to be cleaned up. This code is hit for some non-accelerated cases like list box scrolling.
Attachments
Patch (21.48 KB, patch)
2021-12-22 16:32 PST, Simon Fraser (smfr) 		wenson_hsieh: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-10-12 10:15:29 PDT
<rdar://problem/84153262>
Simon Fraser (smfr)
Comment 2 2021-12-20 19:59:00 PST
*** Bug 234495 has been marked as a duplicate of this bug. ***
Simon Fraser (smfr)
Comment 3 2021-12-22 16:32:00 PST
Created attachment 447841 [details] Patch
Wenson Hsieh
Comment 4 2021-12-22 16:46:14 PST
Comment on attachment 447841 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=447841&action=review r=mews > Source/WebCore/ChangeLog:18 > + The ScrollAnimatorMac::handleWheelEvent() override now simply class into the base class, "class into" => "calls into"
Simon Fraser (smfr)
Comment 5 2021-12-23 09:20:20 PST
https://trac.webkit.org/changeset/287397/webkit
Note You need to log in before you can comment on or make changes to this bug.