REGRESSION: Horizontal scrolling reversed for tilt/ball wheels

• Component: WebKit Misc.
• Status: RESOLVED FIXED

[Peter Kasting]

r25803 made horizontal scrolling direction be correct for shift + vertical wheel and wrong for horizontal wheel (previously, it was the opposite).  This regressed rdar://4965715 .

This has to do with the various coordinate systems for different kinds of horizontal scrolling:

WM_MOUSEWHEEL is positive up, negative down.  Thus shift + wheel is positive left, negative right.
WM_MOUSEHWHEEL is positive right, negative left.
WM_HSCROLL gives you a positive wheel delta along with SB_{LINE,PAGE}{RIGHT,LEFT}.
WebKit expects positive left, negative right.

So, WM_MOUSEHWHEEL needs its delta inverted, whereas shift + WM_MOUSEWHEEL does not, and WM_HSCROLL (if Safari handled it, which it doesn't) simply needs to be set correctly.

I can patch this.  It will involve a few modifications to the WebKit/win code.

[Peter Kasting]

Created attachment 28485 [details]
patch v1

[Darin Fisher (:fishd, Google)]

Comment on attachment 28485 [details]
patch v1

>Index: WebCore/platform/win/WheelEventWin.cpp
...
>+    if (isMouseHWheel)
>+      delta = -delta;  // Windows is <-- -/+ -->, WebKit wants <-- +/- -->

nit: indentation is wrong

otherwise LGTM.

[Peter Kasting]

Fixed in r41642.