Remove unsupported parameters by the unified gesture detector from Gesture Configuration.

Remove unsupported parameters by the unified gesture detector from Gesture Configuration.

There are some parameters in the Gesture Configuration which are
no longer used for unified gesture detector:
max_radius_;
min_flick_speed_squared_
min_rail_break_velocity_
min_scroll_delta_squared_
min_touch_down_duration_in_seconds_for_click_
rail_break_proportion_
rail_start_proportion_
scroll_prediction_seconds_
points_buffered_for_velocity_

BUG=380131
Committed: https://crrev.com/a2349a8f0fa116ff0794d48fefc0074312b2f126
Cr-Commit-Position: refs/heads/master@{#294975}

[lanwei, 2014-09-12 16:20:10]

lanwei@chromium.org changed reviewers:
+ tdresser@chromium.org

[tdresser, 2014-09-12 17:00:02]

https://codereview.chromium.org/570633002/diff/1/ui/events/gestures/gesture_c...
File ui/events/gestures/gesture_configuration.cc (left):

https://codereview.chromium.org/570633002/diff/1/ui/events/gestures/gesture_c...
ui/events/gestures/gesture_configuration.cc:43: int
GestureConfiguration::points_buffered_for_velocity_ = 8;
points_buffered_for_velocity_ isn't used anywhere, is it?

(Are there any others?)

[lanwei, 2014-09-12 18:36:47]

On 2014/09/12 17:00:02, tdresser wrote:
>
https://codereview.chromium.org/570633002/diff/1/ui/events/gestures/gesture_c...
> File ui/events/gestures/gesture_configuration.cc (left):
> 
>
https://codereview.chromium.org/570633002/diff/1/ui/events/gestures/gesture_c...
> ui/events/gestures/gesture_configuration.cc:43: int
> GestureConfiguration::points_buffered_for_velocity_ = 8;
> points_buffered_for_velocity_ isn't used anywhere, is it?
> 
> (Are there any others?)

Deleted points_buffered_for_velocity_, I checked other parameters are currently
in use.

Thank you.

[tdresser, 2014-09-12 18:39:58]

Thanks for the cleanup!

LGTM.
Add points_buffered_for_velocity_ to the CL description.

https://codereview.chromium.org/570633002/diff/20001/ui/aura/gestures/gesture...
File ui/aura/gestures/gesture_recognizer_unittest.cc (left):

https://codereview.chromium.org/570633002/diff/20001/ui/aura/gestures/gesture...
ui/aura/gestures/gesture_recognizer_unittest.cc:503: 
Might as well leave this space here.

[lanwei, 2014-09-12 18:45:38]

lanwei@chromium.org changed reviewers:
+ sadrul@chromium.org

[lanwei, 2014-09-12 18:47:50]

Hello Sadrul,
Can you please take a look at ui/aura/test/aura_test_base.cc,
I removed some unused parameters from it?
Thank you.

https://codereview.chromium.org/570633002/diff/20001/ui/aura/gestures/gesture...
File ui/aura/gestures/gesture_recognizer_unittest.cc (left):

https://codereview.chromium.org/570633002/diff/20001/ui/aura/gestures/gesture...
ui/aura/gestures/gesture_recognizer_unittest.cc:503: 
On 2014/09/12 18:39:57, tdresser wrote:
> Might as well leave this space here.

Done.

[sadrul, 2014-09-15 13:37:42]

lgtm

[lanwei, 2014-09-16 00:29:55]

The CQ bit was checked by lanwei@chromium.org

[commit-bot: I haz the power, 2014-09-16 00:37:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/570633002/40001

[commit-bot: I haz the power, 2014-09-16 02:11:25]

Committed patchset #3 (id:40001) as 919c3d6e882cf2e1429b5074c237c04704397e6a

[commit-bot: I haz the power, 2014-09-16 02:13:27]

Patchset 3 (id:??) landed as
https://crrev.com/a2349a8f0fa116ff0794d48fefc0074312b2f126
Cr-Commit-Position: refs/heads/master@{#294975}