gestures: Generate only either scroll-end or fling-start events at the end of a scroll gesture.

gestures: Generate only either scroll-end or fling-start events at the end of a scroll gesture.

When a fling happens at the end of a scroll, instead of generating both a scroll-end
and a fling-start event, generate only fling-start event. Unfortunately, WebKit still
requires an explicit scroll-end gesture before a fling, so continue to do that as a
special case.

BUG=141460, 141309
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=150852

[xiyuan, 2012-08-08 20:38:49]

Awesome. It works great. Both touch scroll and fling works as expected in app
list.

Thanks for fixing this.

[sadrul, 2012-08-09 00:21:58]

rjkroege@ Please review content/ and gestures/

sky@ Please review ash/ content/ and views/

xiyuan@ Please review app_list/

[sky, 2012-08-09 00:43:16]

LGTM

[xiyuan, 2012-08-09 16:21:27]

app_list/ LGTM

[rjkroege, 2012-08-09 16:51:06]

gr changes LGTM.

http://codereview.chromium.org/10826209/diff/20/content/browser/renderer_host...
File content/browser/renderer_host/render_widget_host_view_aura.cc (right):

http://codereview.chromium.org/10826209/diff/20/content/browser/renderer_host...
content/browser/renderer_host/render_widget_host_view_aura.cc:1322: // WebKit
requires that a scroll ends first before a fling starts.
It is planned to remove this requirement on WebKit. Did you test without the
presence of this code? It is my intent that this ought to be removed from here
if it is currently necessary.

http://codereview.chromium.org/10826209/diff/20/content/browser/renderer_host...
File content/browser/renderer_host/render_widget_host_view_win.cc (right):

http://codereview.chromium.org/10826209/diff/20/content/browser/renderer_host...
content/browser/renderer_host/render_widget_host_view_win.cc:2887: // WebKit
requires that a scroll ends first before a fling starts.
same comment. :-)

[sadrul, 2012-08-09 17:44:02]

http://codereview.chromium.org/10826209/diff/20/content/browser/renderer_host...
File content/browser/renderer_host/render_widget_host_view_aura.cc (right):

http://codereview.chromium.org/10826209/diff/20/content/browser/renderer_host...
content/browser/renderer_host/render_widget_host_view_aura.cc:1322: // WebKit
requires that a scroll ends first before a fling starts.
On 2012/08/09 16:51:06, rjkroege wrote:
> It is planned to remove this requirement on WebKit. Did you test without the
> presence of this code? It is my intent that this ought to be removed from here
> if it is currently necessary.

Nice! My impression was that it was still necessary. But I tried without this
block of code, in debug and release builds on desktop, and on device, and scroll
with/without fling seems to work fine. So I have removed this change. Thanks!