[PATCH 2/11] ozone: evdev: Move KeyboardEvdev usage during dispatch to EventFactoryEvdev

ui/events/ozone/evdev/event_dispatch_callback.h

Index: ui/events/ozone/evdev/event_dispatch_callback.h
diff --git a/ui/events/ozone/evdev/event_dispatch_callback.h b/ui/events/ozone/evdev/event_dispatch_callback.h
index efbb8b4ef613b5c3e7c2462723077131e81d989c..8a478a8a811fa938fef7979f6ae6d4b6ffcab676 100644
--- a/ui/events/ozone/evdev/event_dispatch_callback.h
+++ b/ui/events/ozone/evdev/event_dispatch_callback.h
@@ -26,6 +26,9 @@ class Event;
 
 typedef base::Callback<void(scoped_ptr<Event>)> EventDispatchCallback;
 
+typedef base::Callback<void(int device_id, unsigned int code, bool down)>

[alexst (slow to review), 2015/01/26 22:18:25]

My brain wants more structure here. 

You are passing things individually in one place, but as a struct in another. I
know there is an interface down the road that replaces these, but similarly, it
splits between the two styles.

I gather it's a callback param limit thing, but since we are bumping against it
in one place, would you hate me if I asked we moved all of these to structs?

[spang, 2015/01/26 23:01:21]

We could remove the structs once the 7 argument limit is removed. There is a
patch for this in review.

Why make more structs just to remove them?

[spang, 2015/01/27 21:31:54]

Done.

+    KeyEventDispatchCallback;
+
 struct TouchEventParams {
   TouchEventParams(int device_id,
                    int touch_id,