[chromium] Simplify fling handling
Created attachment 176045 [details] Patch
On ChromeOS we want the synthetic wheel behavior. On Android you do not, but that doesn't mean you can just delete the other path.
Comment on attachment 176045 [details] Patch I think you want to pick between wheel-like and touch-like flings based on the input event's SourceDevice. If it's touchpad, do the wheel fling. If it's touchscreen, do a touch fling.
Robert, could you clarify what parts of this behavior you actually want to keep on ChromeOS and why? I don't know what's intended behavior and what's historical cruft, so I erred on the side of deletion.
(In reply to comment #4) > Robert, could you clarify what parts of this behavior you actually want to keep on ChromeOS and why? I don't know what's intended behavior and what's historical cruft, so I erred on the side of deletion. The code currently here implements touchpad-style fling (for lack of a better term). It's what you get on a mac - a sequence of wheel events targeting a specific point on the page (not an element). The sort of fling you want in android (which I'll call touchscreen fling) is different - there you get a sequence of scrolls (not events) targeting a specific scrollable layer (not a point).
Created attachment 176093 [details] Patch
PTAL, Imade the patch a no-op for touchpad flings.
Comment on attachment 176093 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=176093&action=review Could you update WebCompositorInputHandlerImplTest.cpp? It has several tests for the touchpad behavior - it would be good to have some for the touchscreen behavior as well. > Source/WebKit/chromium/src/WebCompositorInputHandlerImpl.cpp:320 > + TRACE_EVENT2("cc", "WebCompositorInputHandlerImpl::scrollBy", "x", increment.x, "y", increment.y); i think this should be in the "webkit" trace category, not "cc" (the other ones in this file are wrong too, methinks)
Created attachment 176123 [details] Patch
Added tests for touchscreen fling.
Comment on attachment 176123 [details] Patch R=me
Comment on attachment 176123 [details] Patch Rejecting attachment 176123 [details] from commit-queue. New failing tests: inspector-protocol/debugger-terminate-dedicated-worker-while-paused.html Full output: http://queues.webkit.org/results/15003371
Comment on attachment 176123 [details] Patch Clearing flags on attachment: 176123 Committed r135808: <http://trac.webkit.org/changeset/135808>
All reviewed patches have been landed. Closing bug.