[PATCH 8/11] ozone: evdev: Plumb device changes to EventFactoryEvdev

ozone: evdev: Plumb device changes to EventFactoryEvdev

This is part of a series moving IO on evdev devices to a fast thread,
since UI thread is too slow for the cursor.

We must access DeviceDataManager from the UI thread only, so send a new
list to EventFactoryEvdev when devices are added or removed, and update
DeviceDataManager from there.

InputControllerEvdev also needs to know if there's a mouse or touchpad
attached, so additionally plumb lists for those devices and update this
property on InputControllerEvdev.

BUG=449710
TEST=boot link_freon & move mouse

Committed: https://crrev.com/c77b18bf9bee62805a9ed5d1ae12e2f0c6944f45
Cr-Commit-Position: refs/heads/master@{#313865}

[spang, 2015-01-23 22:00:48]

spang@chromium.org changed reviewers:
+ alexst@chromium.org

[alexst (slow to review), 2015-01-28 16:16:44]

lgtm

https://codereview.chromium.org/868043003/diff/20001/ui/events/ozone/evdev/de...
File ui/events/ozone/evdev/device_event_dispatcher_evdev.h (right):

https://codereview.chromium.org/868043003/diff/20001/ui/events/ozone/evdev/de...
ui/events/ozone/evdev/device_event_dispatcher_evdev.h:62: class
DeviceEventDispatcherEvdev {
Please add some threading notes.

https://codereview.chromium.org/868043003/diff/20001/ui/events/ozone/evdev/in...
File ui/events/ozone/evdev/input_controller_evdev.cc (right):

https://codereview.chromium.org/868043003/diff/20001/ui/events/ozone/evdev/in...
ui/events/ozone/evdev/input_controller_evdev.cc:33: void
InputControllerEvdev::SetHasMouse(bool has_mouse) {
should_this_be_hacker_style()?

Here and for all the other trivial getters and setters?

[spang, 2015-01-28 19:04:54]

https://codereview.chromium.org/868043003/diff/20001/ui/events/ozone/evdev/de...
File ui/events/ozone/evdev/device_event_dispatcher_evdev.h (right):

https://codereview.chromium.org/868043003/diff/20001/ui/events/ozone/evdev/de...
ui/events/ozone/evdev/device_event_dispatcher_evdev.h:62: class
DeviceEventDispatcherEvdev {
On 2015/01/28 16:16:44, alexst wrote:
> Please add some threading notes.

To #11, there's no thread until.

https://codereview.chromium.org/868043003/diff/20001/ui/events/ozone/evdev/in...
File ui/events/ozone/evdev/input_controller_evdev.cc (right):

https://codereview.chromium.org/868043003/diff/20001/ui/events/ozone/evdev/in...
ui/events/ozone/evdev/input_controller_evdev.cc:33: void
InputControllerEvdev::SetHasMouse(bool has_mouse) {
On 2015/01/28 16:16:44, alexst wrote:
> should_this_be_hacker_style()?
> 
> Here and for all the other trivial getters and setters?

Will change as a followup, don't want to add a large number of style changes in
this patch.

[spang, 2015-01-30 03:01:07]

The CQ bit was checked by spang@chromium.org

[commit-bot: I haz the power, 2015-01-30 03:01:26]

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

[commit-bot: I haz the power, 2015-01-30 03:42:46]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-01-30 03:43:42]

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