Prevent fling gesture if most recent touch move was consumed.

Prevent fling gesture if most recent touch move was consumed.

BUG=361685
TEST=TouchDispositionFilterTest.ConsumedScrollUpdateMakesFlingScrollEnd

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=262823

[tdresser, 2014-04-09 18:16:49]

PTAL, thanks.

[jdduke (slow), 2014-04-09 18:25:45]

lgtm

https://codereview.chromium.org/231333003/diff/1/ui/events/gesture_detection/...
File ui/events/gesture_detection/touch_disposition_gesture_filter_unittest.cc
(right):

https://codereview.chromium.org/231333003/diff/1/ui/events/gesture_detection/...
ui/events/gesture_detection/touch_disposition_gesture_filter_unittest.cc:887:
PushGesture(ET_SCROLL_FLING_CANCEL);
I'm not aware of any legitimate cases where GFS and GFC would be bundled in the
same packet, but I guess it doesn't hurt here.

https://codereview.chromium.org/231333003/diff/1/ui/events/gesture_detection/...
ui/events/gesture_detection/touch_disposition_gesture_filter_unittest.cc:893: }
Could you add the start of a new touch sequence and verify that we don't get
FLING_CANCEL?

[tdresser, 2014-04-09 18:33:19]

Thanks for the speedy review.

https://codereview.chromium.org/231333003/diff/1/ui/events/gesture_detection/...
File ui/events/gesture_detection/touch_disposition_gesture_filter_unittest.cc
(right):

https://codereview.chromium.org/231333003/diff/1/ui/events/gesture_detection/...
ui/events/gesture_detection/touch_disposition_gesture_filter_unittest.cc:893: }
On 2014/04/09 18:25:45, jdduke wrote:
> Could you add the start of a new touch sequence and verify that we don't get
> FLING_CANCEL?

Done.

[tdresser, 2014-04-09 18:33:48]

The CQ bit was checked by tdresser@chromium.org

[commit-bot: I haz the power, 2014-04-09 18:34:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tdresser@chromium.org/231333003/20001

[commit-bot: I haz the power, 2014-04-09 21:33:52]

Change committed as 262823