Add scroll threshold for app list for mouse wheel events.

Add scroll threshold for app list for mouse wheel events.

This CL fixes an issue with touchpad events being read as mousewheel
events and causing jank on touchpad scroll in the app list. Instead of
figuring out the whole stack and why the wrong event is coming in, just
use the scroll threshold everywhere.

BUG=594264
Committed: https://crrev.com/4db974d7763416334efdbe4859427475ff5889f9
Cr-Commit-Position: refs/heads/master@{#397642}

[calamity, 2016-06-02 08:40:46]

calamity@chromium.org changed reviewers:
+ mgiuca@chromium.org

[calamity, 2016-06-02 08:40:55]

Patchset #1 (id:1) has been deleted

[Matt Giuca, 2016-06-02 08:44:18]

https://codereview.chromium.org/2029993002/diff/40001/ui/app_list/pagination_...
File ui/app_list/pagination_controller.cc (right):

https://codereview.chromium.org/2029993002/diff/40001/ui/app_list/pagination_...
ui/app_list/pagination_controller.cc:42: // Do not scroll on very small events.
// TODO(calamity): Something something https://crbug.com/594264.

[Matt Giuca, 2016-06-02 08:44:41]

lgtm with above nit

[calamity, 2016-06-03 01:25:29]

The CQ bit was checked by calamity@chromium.org

[calamity, 2016-06-03 01:25:30]

The patchset sent to the CQ was uploaded after l-g-t-m from mgiuca@chromium.org
Link to the patchset: https://codereview.chromium.org/2029993002/#ps60001
(title: "TODO - I bless the rains down in Africa")

[calamity, 2016-06-03 01:25:45]

https://codereview.chromium.org/2029993002/diff/40001/ui/app_list/pagination_...
File ui/app_list/pagination_controller.cc (right):

https://codereview.chromium.org/2029993002/diff/40001/ui/app_list/pagination_...
ui/app_list/pagination_controller.cc:42: // Do not scroll on very small events.
On 2016/06/02 08:44:18, Matt Giuca wrote:
> // TODO(calamity): Something something https://crbug.com/594264.

Done.

[commit-bot: I haz the power, 2016-06-03 01:26:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2029993002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2029993002/60001

[Matt Giuca, 2016-06-03 03:43:53]

I don't get this patch set name.

[commit-bot: I haz the power, 2016-06-03 05:43:13]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-06-03 05:44:49]

Description was changed from

==========
Add scroll threshold for app list for mouse wheel events.

This CL fixes an issue with touchpad events being read as mousewheel
events and causing jank on touchpad scroll in the app list. Instead of
figuring out the whole stack and why the wrong event is coming in, just
use the scroll threshold everywhere.

BUG=594264
==========

to

==========
Add scroll threshold for app list for mouse wheel events.

This CL fixes an issue with touchpad events being read as mousewheel
events and causing jank on touchpad scroll in the app list. Instead of
figuring out the whole stack and why the wrong event is coming in, just
use the scroll threshold everywhere.

BUG=594264

Committed: https://crrev.com/4db974d7763416334efdbe4859427475ff5889f9
Cr-Commit-Position: refs/heads/master@{#397642}
==========

[commit-bot: I haz the power, 2016-06-03 05:44:50]

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