Bug 228572 – REGRESSION(r279564): [GTK] Mouse wheel scrolling scrolls too little (take two)

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Michael Catanzaro]

This is a repeat of bug #224686, we broke mouse wheel scroll speed again. I've started bisecting and am currently at r279963 as first known bad commit and r279541 as last known good. I see a few scrolling commits within this range, and will probably find the culprit when I continue bisecting tomorrow.

[Michael Catanzaro]

I'm almost done bisecting and it looks like the bad commit is probably r279564 "[css-scroll-snap] Triggering a layout during scroll causes jittery scrolling on Mac when dragging the scrollbar." Hi Martin, do you have time to take a look?

[Martin Robinson]

Yeah, I can take a look at this tomorrow or Monday. Thanks for the bisection.

[Martin Robinson]

This problem shows up when you scroll rapidly with scroll wheel and with smooth scrolling turned on. The issue here seems to be that when ScrollAnimator handles a wheel event, it calculates the destination based on the current position, rather than the current target position for the running animation.

r279564 uncovered this issue by fixing a bug in ScrollAnimationSmooth that disregarded the requested position in favor of a bogus one, but also making the scrolling by delta relative to the current position (rather than the desired one).

This effectively means that the faster you scroll, the slower WebKit scrolls. This is, to say the least, wrong.

[Martin Robinson]

Created attachment 434617 [details]
Patch

[Michael Catanzaro]

Comment on attachment 434617 [details]
Patch

Nice test!

[EWS]

Committed r280515 (240147@main): <https://commits.webkit.org/240147@main>

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