2009-12-02 Evan Stade <estade@chromium.org>
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Dec 2009 23:21:55 +0000 (23:21 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Dec 2009 23:21:55 +0000 (23:21 +0000)
        Reviewed by Darin Fisher.

        https://bugs.webkit.org/show_bug.cgi?id=32029
        Mousewheel event delta has reversed sign on Linux Chrome.

        No new tests (layout tests don't cover event conversion code).
        Manual test: http://www.switchonthecode.com/tutorials/javascript-tutorial-the-scroll-wheel

        * src/gtk/WebInputEventFactory.cpp:
        (WebKit::WebInputEventFactory::mouseWheelEvent):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@51614 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/chromium/ChangeLog
WebKit/chromium/src/gtk/WebInputEventFactory.cpp

index 487b6011a6023f5d1e6ba44f639660090489bfd0..1da877845b8bba03869278b6ce8a13a990cdaf98 100644 (file)
@@ -1,3 +1,16 @@
+2009-12-02  Evan Stade  <estade@chromium.org>
+
+        Reviewed by Darin Fisher.
+
+        https://bugs.webkit.org/show_bug.cgi?id=32029
+        Mousewheel event delta has reversed sign on Linux Chrome.
+
+        No new tests (layout tests don't cover event conversion code).
+        Manual test: http://www.switchonthecode.com/tutorials/javascript-tutorial-the-scroll-wheel
+
+        * src/gtk/WebInputEventFactory.cpp:
+        (WebKit::WebInputEventFactory::mouseWheelEvent):
+
 2009-12-02  Anton Muhin  <antonm@chromium.org>
 
         Reviewed by Adam Barth.
index 449dee11dac78b83251d42751c55fd6af3811307..7125a16f499c05fa14064e715fdfcad8baf1c52e 100644 (file)
@@ -482,11 +482,11 @@ WebMouseWheelEvent WebInputEventFactory::mouseWheelEvent(const GdkEventScroll* e
         break;
     case GDK_SCROLL_LEFT:
         result.deltaX = scrollbarPixelsPerTick;
-        result.wheelTicksX = -1;  // Match Windows positive/negative orientation
+        result.wheelTicksX = 1;
         break;
     case GDK_SCROLL_RIGHT:
         result.deltaX = -scrollbarPixelsPerTick;
-        result.wheelTicksX = 1;
+        result.wheelTicksX = -1;
         break;
     }