GestureScrollEnd event had non-zero velocity causing overscroll on software fling path.

GestureScrollEnd event had non-zero velocity causing overscroll on software fling path.

GestureScrollEnd (indicating the end of a touchscreen interactive scrolling 
gesture) contained velocity. This was causing an additional scroll at the start of
a touchscreen-initiated fling.

BUG=128537
TEST=by hand


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=139943

[sadrul, 2012-05-31 15:19:23]

LGTM

https://chromiumcodereview.appspot.com/10446103/diff/2001/ui/base/gestures/ge...
File ui/base/gestures/gesture_sequence.cc (right):

https://chromiumcodereview.appspot.com/10446103/diff/2001/ui/base/gestures/ge...
ui/base/gestures/gesture_sequence.cc:499: float velocity_scaling  = 1. / 900.;
const?

[rjkroege, 2012-05-31 15:26:46]

https://chromiumcodereview.appspot.com/10446103/diff/2001/ui/base/gestures/ge...
File ui/base/gestures/gesture_sequence.cc (right):

https://chromiumcodereview.appspot.com/10446103/diff/2001/ui/base/gestures/ge...
ui/base/gestures/gesture_sequence.cc:499: float velocity_scaling  = 1. / 900.;
On 2012/05/31 15:19:23, sadrul wrote:
> const?

done

[commit-bot: I haz the power, 2012-05-31 15:27:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rjkroege@chromium.org/10446103/6002

[commit-bot: I haz the power, 2012-05-31 15:46:10]

Try job failure for 10446103-6002 (retry) on win for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...

[commit-bot: I haz the power, 2012-05-31 19:24:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rjkroege@chromium.org/10446103/7004

[commit-bot: I haz the power, 2012-05-31 19:24:53]

Presubmit check for 10446103-7004 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    ui/aura

[rjkroege, 2012-05-31 19:29:32]

+sky for owners in ui/aura/gestures.

[sky, 2012-05-31 20:57:21]

LGTM

[commit-bot: I haz the power, 2012-05-31 21:03:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rjkroege@chromium.org/10446103/7004

[commit-bot: I haz the power, 2012-06-01 01:08:53]

Change committed as 139943