chromeos: Add ArcInputBridge to components/arc

components/arc/arc_bridge_input_devices.h

+#ifndef CHROMEOS_ARC_BRIDGE_INPUT_DEVICES_H_

[jochen (gone - plz use gerrit), 2015/11/17 15:03:44]

COMPONENTS_ARC_ARC_BRIDGE_INPUT_DEVICES_H_

also, missing copyright header

[denniskempin, 2015/11/19 19:22:08]

Done. respective macro on new file in exo

+#define CHROMEOS_ARC_BRIDGE_INPUT_DEVICES_H_
+
+#include "base/files/scoped_file.h"
+#include "base/macros.h"
+#include "base/memory/scoped_vector.h"
+#include "ui/events/event.h"
+#include "ui/events/event_handler.h"
+
+#include <linux/types.h>
+
+namespace arc {
+
+class BridgeInputDevice : public ui::EventHandler {
+ public:
+  BridgeInputDevice(base::ScopedFD fd);

[jochen (gone - plz use gerrit), 2015/11/17 15:03:44]

explicit

please add a virtual dtor

[denniskempin, 2015/11/19 19:22:08]

Done.

+
+  // Send input_event through file descriptor.

[jochen (gone - plz use gerrit), 2015/11/17 15:03:44]

please document what type code and value are

[denniskempin, 2015/11/19 19:22:08]

Done.

+  void SendEvent(base::TimeDelta time, __u16 type, __u16 code, __s32 value);

[Luis Héctor Chávez, 2015/11/17 17:51:53]

Can you use types from stdint instead? I know we're linux-only, but no other
part of the codebase seems to be using these types.

[denniskempin, 2015/11/19 19:22:08]

Done. It's the types used by the struct definition, but at some point we are
implicitly type casting anyway, so that's ok. 

+
+  // Shorthand for sending a SYN_REPORT input_event.
+  void SendSynReport(base::TimeDelta time);
+
+ private:
+  base::ScopedFD fd_;
+};

[jochen (gone - plz use gerrit), 2015/11/17 15:03:44]

disallow copy/assign

[denniskempin, 2015/11/19 19:22:08]

Done.

+
+// BridgeInputDevice for handling keyboard events
+class KeyboardBridgeInputDevice : public BridgeInputDevice {
+  // input_event values for keyboard events
+  static const int kKeyReleased = 0;

[jochen (gone - plz use gerrit), 2015/11/17 15:03:44]

please move purely private constants to the cc file

[denniskempin, 2015/11/19 19:22:08]

Done.

+  static const int kKeyPressed = 1;
+  static const int kKeyRepeated = 2;
+
+ public:
+  KeyboardBridgeInputDevice(base::ScopedFD fd);

[jochen (gone - plz use gerrit), 2015/11/17 15:03:44]

explicit / dtor / disallow copy & assign - please review the chromium C++ style
guide

[denniskempin, 2015/11/19 19:22:08]

Done.

+  void OnKeyEvent(ui::KeyEvent* event) override;
+};
+
+// BridgeInputDevice for handling mouse events
+class MouseBridgeInputDevice : public BridgeInputDevice {
+ public:
+  MouseBridgeInputDevice(base::ScopedFD fd);
+  void OnMouseEvent(ui::MouseEvent* event) override;
+
+ private:
+  void SendMouseButton(ui::MouseEvent* event,
+                       int flag,
+                       int evdev_code,
+                       bool button_value);
+};
+
+// BridgeDevice for handling touch events
+class TouchscreenBridgeInputDevice : public BridgeInputDevice {
+  static const int kEmptySlot = -1;
+  static const int kMaxSlots = 32;
+
+ public:
+  TouchscreenBridgeInputDevice(base::ScopedFD fd);
+  virtual void OnTouchEvent(ui::TouchEvent* event) override;
+
+ private:
+  // Manages available slot id's and returns slot assigned to this touch event.
+  int AcquireSlot(ui::TouchEvent* event);
+  // Locates slot for tracking id
+  int FindSlot(int tracking_id);
+
+  std::vector<int> current_slot_tracking_ids_;
+  int current_slot_;
+};
+}
+
+#endif  // CHROMEOS_ARC_BRIDGE_INPUT_DEVICES_H_