[PATCH 1/11] ozone: evdev: Move touch ui::Event transformation to EventFactoryEvdev

ozone: evdev: Move touch ui::Event transformation 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.

DeviceDataManager is a UI thread object, so move final transformation
of touches into EventFactoryEvdev prior to dispatch into UI. This
will avoid sharing state across threads.

BUG=449710
TEST=boot link_freon & move cursor

Committed: https://crrev.com/40ee4d543a2f09bf72e19f5db26e2f20d52e344c
Cr-Commit-Position: refs/heads/master@{#313204}

[spang, 2015-01-23 21:56:18]

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

[alexst (slow to review), 2015-01-26 18:20:13]

https://codereview.chromium.org/874713002/diff/40001/ui/events/ozone/evdev/ev...
File ui/events/ozone/evdev/event_factory_evdev.cc (right):

https://codereview.chromium.org/874713002/diff/40001/ui/events/ozone/evdev/ev...
ui/events/ozone/evdev/event_factory_evdev.cc:209: // Transform the event
according (this is used to align touches
accordingly?

https://codereview.chromium.org/874713002/diff/40001/ui/events/ozone/evdev/ev...
ui/events/ozone/evdev/event_factory_evdev.cc:223:
PostUiEvent(touch_event.Pass());
You can skip the post task and the scoped_ptr here.

[alexst (slow to review), 2015-01-26 22:08:16]

lgtm

[spang, 2015-01-26 23:43:35]

https://codereview.chromium.org/874713002/diff/40001/ui/events/ozone/evdev/ev...
File ui/events/ozone/evdev/event_factory_evdev.cc (right):

https://codereview.chromium.org/874713002/diff/40001/ui/events/ozone/evdev/ev...
ui/events/ozone/evdev/event_factory_evdev.cc:209: // Transform the event
according (this is used to align touches
On 2015/01/26 18:20:13, alexst wrote:
> accordingly?

Done.

https://codereview.chromium.org/874713002/diff/40001/ui/events/ozone/evdev/ev...
ui/events/ozone/evdev/event_factory_evdev.cc:223:
PostUiEvent(touch_event.Pass());
On 2015/01/26 18:20:13, alexst wrote:
> You can skip the post task and the scoped_ptr here.

Yep but will need to wait until patch #9 ("Add a device event dispatcher that
forwards to UI thread").

[spang, 2015-01-27 01:08:26]

The CQ bit was checked by spang@chromium.org

[commit-bot: I haz the power, 2015-01-27 01:09:01]

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

[commit-bot: I haz the power, 2015-01-27 01:57:53]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-01-27 02:06:26]

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