Removed DeviceDataManagerX11::scrollclass_devices_

Removed DeviceDataManagerX11::scrollclass_devices_

This field isn't needed, and goes against the idea that a mouse either supports
scroll classes entirely or not at all. Instead we should use logic that checks
whether a device supports horizontal, vertical or both scroll classes. I also
fixed some logic that could have passed an invalid validator ID into
XIMaskIsSet---this new check covers the check removed above it.

BUG=596013
Committed: https://crrev.com/b89e8ac7021fcee56c7cc34fd077eac253c99d0b
Cr-Commit-Position: refs/heads/master@{#388208}

[Will Shackleton, 2016-04-15 17:04:52]

Description was changed from

==========
Removed DeviceDataManagerX11::scrollclass_devices_

This field isn't needed, and goes against the idea that a mouse either supports
scroll classes entirely or not at all. Instead we should use logic that checks
whether a device supports horizontal, vertical or both scroll classes.

BUG=596013
==========

to

==========
Removed DeviceDataManagerX11::scrollclass_devices_

This field isn't needed, and goes against the idea that a mouse either supports
scroll classes entirely or not at all. Instead we should use logic that checks
whether a device supports horizontal, vertical or both scroll classes.

BUG=596013
==========

[Will Shackleton, 2016-04-15 17:04:52]

w.shackleton@gmail.com changed reviewers:
+ bokan@chromium.org, dtapuska@chromium.org, sadrul@chromium.org

[Will Shackleton, 2016-04-15 17:06:42]

Description was changed from

==========
Removed DeviceDataManagerX11::scrollclass_devices_

This field isn't needed, and goes against the idea that a mouse either supports
scroll classes entirely or not at all. Instead we should use logic that checks
whether a device supports horizontal, vertical or both scroll classes.

BUG=596013
==========

to

==========
Removed DeviceDataManagerX11::scrollclass_devices_

This field isn't needed, and goes against the idea that a mouse either supports
scroll classes entirely or not at all. Instead we should use logic that checks
whether a device supports horizontal, vertical or both scroll classes. I also
fixed some logic that could have passed an invalid validator ID into
XIMaskIsSet---this new check covers the check removed above it.

BUG=596013
==========

[sadrul, 2016-04-19 14:25:25]

lgtm

[Will Shackleton, 2016-04-19 14:28:51]

The CQ bit was checked by w.shackleton@gmail.com

[commit-bot: I haz the power, 2016-04-19 14:29:09]

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

[commit-bot: I haz the power, 2016-04-19 15:14:55]

Description was changed from

==========
Removed DeviceDataManagerX11::scrollclass_devices_

This field isn't needed, and goes against the idea that a mouse either supports
scroll classes entirely or not at all. Instead we should use logic that checks
whether a device supports horizontal, vertical or both scroll classes. I also
fixed some logic that could have passed an invalid validator ID into
XIMaskIsSet---this new check covers the check removed above it.

BUG=596013
==========

to

==========
Removed DeviceDataManagerX11::scrollclass_devices_

This field isn't needed, and goes against the idea that a mouse either supports
scroll classes entirely or not at all. Instead we should use logic that checks
whether a device supports horizontal, vertical or both scroll classes. I also
fixed some logic that could have passed an invalid validator ID into
XIMaskIsSet---this new check covers the check removed above it.

BUG=596013
==========

[commit-bot: I haz the power, 2016-04-19 15:14:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:12:17]

Description was changed from

==========
Removed DeviceDataManagerX11::scrollclass_devices_

This field isn't needed, and goes against the idea that a mouse either supports
scroll classes entirely or not at all. Instead we should use logic that checks
whether a device supports horizontal, vertical or both scroll classes. I also
fixed some logic that could have passed an invalid validator ID into
XIMaskIsSet---this new check covers the check removed above it.

BUG=596013
==========

to

==========
Removed DeviceDataManagerX11::scrollclass_devices_

This field isn't needed, and goes against the idea that a mouse either supports
scroll classes entirely or not at all. Instead we should use logic that checks
whether a device supports horizontal, vertical or both scroll classes. I also
fixed some logic that could have passed an invalid validator ID into
XIMaskIsSet---this new check covers the check removed above it.

BUG=596013

Committed: https://crrev.com/b89e8ac7021fcee56c7cc34fd077eac253c99d0b
Cr-Commit-Position: refs/heads/master@{#388208}
==========

[commit-bot: I haz the power, 2016-04-22 19:12:18]

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