Fixed first step of scrolling not applying

Fixed first step of scrolling not applying

I fixed the first step of scrolling not applying when switching from one slave
mouse device to another. This bug was happening because the Xinput2 event
XIDeviceChangedEvent is used for two different reasons, a device's properties
changing and the current slave device being switched. This code checks that the
reason field is set to XIDeviceChange (rather than XISlaveSwitch).

BUG=593453
Committed: https://crrev.com/47c6f3e7252ece9b52e5b86d40944e66442ea7aa
Cr-Commit-Position: refs/heads/master@{#388182}

[Will Shackleton, 2016-04-13 14:29:32]

Description was changed from

==========
Fixed first step of scrolling not applying

I fixed the first step of scrolling not applying when switching from one slave
mouse device to another. This bug was happening because the Xinput2 event
XIDeviceChangedEvent is used for two different reasons, a device's properties
changing and the current slave device being switched. This code checks that the
reason field is set to XIDeviceChange (rather than XISlaveSwitch).

BUG=593453
==========

to

==========
Fixed first step of scrolling not applying

I fixed the first step of scrolling not applying when switching from one slave
mouse device to another. This bug was happening because the Xinput2 event
XIDeviceChangedEvent is used for two different reasons, a device's properties
changing and the current slave device being switched. This code checks that the
reason field is set to XIDeviceChange (rather than XISlaveSwitch).

BUG=593453
==========

[Will Shackleton, 2016-04-13 14:29:32]

w.shackleton@gmail.com changed reviewers:
+ bokan@chromium.org, taviso@google.com

[Will Shackleton, 2016-04-13 14:48:11]

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

[Will Shackleton, 2016-04-15 16:17:58]

Have a look at https://people.freedesktop.org/~whot/xi2-recipes/part3.c lines
131--145 for some other code using this logic. I did find an equivalent piece of
code to what I've added here in GTK+3 as well.

[sadrul, 2016-04-19 01:24:12]

lgtm

[Will Shackleton, 2016-04-19 10:22:17]

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

[commit-bot: I haz the power, 2016-04-19 10:22:33]

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

[commit-bot: I haz the power, 2016-04-19 11:14:07]

Description was changed from

==========
Fixed first step of scrolling not applying

I fixed the first step of scrolling not applying when switching from one slave
mouse device to another. This bug was happening because the Xinput2 event
XIDeviceChangedEvent is used for two different reasons, a device's properties
changing and the current slave device being switched. This code checks that the
reason field is set to XIDeviceChange (rather than XISlaveSwitch).

BUG=593453
==========

to

==========
Fixed first step of scrolling not applying

I fixed the first step of scrolling not applying when switching from one slave
mouse device to another. This bug was happening because the Xinput2 event
XIDeviceChangedEvent is used for two different reasons, a device's properties
changing and the current slave device being switched. This code checks that the
reason field is set to XIDeviceChange (rather than XISlaveSwitch).

BUG=593453
==========

[commit-bot: I haz the power, 2016-04-19 11:14:08]

Committed patchset #1 (id:1)

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

Description was changed from

==========
Fixed first step of scrolling not applying

I fixed the first step of scrolling not applying when switching from one slave
mouse device to another. This bug was happening because the Xinput2 event
XIDeviceChangedEvent is used for two different reasons, a device's properties
changing and the current slave device being switched. This code checks that the
reason field is set to XIDeviceChange (rather than XISlaveSwitch).

BUG=593453
==========

to

==========
Fixed first step of scrolling not applying

I fixed the first step of scrolling not applying when switching from one slave
mouse device to another. This bug was happening because the Xinput2 event
XIDeviceChangedEvent is used for two different reasons, a device's properties
changing and the current slave device being switched. This code checks that the
reason field is set to XIDeviceChange (rather than XISlaveSwitch).

BUG=593453

Committed: https://crrev.com/47c6f3e7252ece9b52e5b86d40944e66442ea7aa
Cr-Commit-Position: refs/heads/master@{#388182}
==========

[commit-bot: I haz the power, 2016-04-19 11:15:12]

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