[webkit-reviews] review granted: [Bug 17589] Scroll wheel sensitivity ignored : [Attachment 22284] Corrected patch

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has granted Maxime Britto <britto at apple.com>'s
request for review:
Bug 17589: Scroll wheel sensitivity ignored
https://bugs.webkit.org/show_bug.cgi?id=17589

Attachment 22284: Corrected patch
https://bugs.webkit.org/attachment.cgi?id=22284&action=edit

------- Additional Comments from Eric Seidel <eric at webkit.org>
Looks great!

I suggest you make:
if (deltaX)

part of the 
scrollAndAcceptEvent inline
as an early return:

if (!delta)
   return;

Also, there are some spacing issues here:

+static inline void adjustDeltaForPageScrollMode(float& delta, bool
pageScrollEnabled, int visibleWidthOrHeight)
+{
+    if (pageScrollEnabled)
+	     delta = (delta > 0 ? visibleWidthOrHeight : -visibleWidthOrHeight)
/ LINE_STEP_WIN;
+}

Land away!