Changeset 112363 in webkit

Timestamp:

Mar 27, 2012 9:27:18 PM (12 years ago)

Author:

commit-queue@webkit.org

Message:

[chromium] Send wheel events to main thread even if we think nothing is scrollable
​https://bugs.webkit.org/show_bug.cgi?id=82408

Patch by James Robinson <​jamesr@chromium.org> on 2012-03-27
Reviewed by Adrienne Walker.

• src/WebCompositorInputHandlerImpl.cpp:

(WebKit::WebCompositorInputHandlerImpl::handleInputEventInternal):

Location:

trunk/Source/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• src/WebCompositorInputHandlerImpl.cpp (modified) (1 diff)

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2012-03-27  James Robinson  <jamesr@chromium.org>
+
+        [chromium] Send wheel events to main thread even if we think nothing is scrollable
+        https://bugs.webkit.org/show_bug.cgi?id=82408
+
+        Reviewed by Adrienne Walker.
+
+        * src/WebCompositorInputHandlerImpl.cpp:
+        (WebKit::WebCompositorInputHandlerImpl::handleInputEventInternal):
+
 2012-03-27  Nat Duca  <nduca@chromium.org>
 

trunk/Source/WebKit/chromium/src/WebCompositorInputHandlerImpl.cpp

@@ -189 +189 @@
         }
         case CCInputHandlerClient::ScrollIgnored:
-            return DropEvent;
+            // FIXME: This should be DropEvent, but in cases where we fail to properly sync scrollability it's safer to send the
+            // event to the main thread. Change back to DropEvent once we have synchronization bugs sorted out.
+            return DidNotHandle; 
         case CCInputHandlerClient::ScrollFailed:
             return DidNotHandle;