Automatically remap Command key on an Apple keyboard to Control [part 2 of 2 - Chrome part]

chrome/browser/chromeos/xinput_hierarchy_changed_event_listener.cc

Index: chrome/browser/chromeos/xinput_hierarchy_changed_event_listener.cc
diff --git a/chrome/browser/chromeos/xinput_hierarchy_changed_event_listener.cc b/chrome/browser/chromeos/xinput_hierarchy_changed_event_listener.cc
index fcab3423f14cfd15a76c4e5154234a1b5ab70129..957708dd715f1ce0bc2721a5750f3cf019f862df 100644
--- a/chrome/browser/chromeos/xinput_hierarchy_changed_event_listener.cc
+++ b/chrome/browser/chromeos/xinput_hierarchy_changed_event_listener.cc
@@ -11,6 +11,7 @@
 #include "chrome/browser/chromeos/input_method/xkeyboard.h"
 #include "ui/base/x/x11_util.h"
 
+namespace chromeos {
 namespace {
 
 // Gets the major opcode for XInput2. Returns -1 on error.
@@ -28,28 +29,37 @@ int GetXInputOpCode() {
   return xi_opcode;
 }
 
-// Starts listening to the XI_HierarchyChanged events.
+// Starts listening to the XI_HierarchyChanged, XI_KeyPress, and XI_KeyRelease
+// events.
 void SelectXInputEvents() {
   XIEventMask evmask;
   unsigned char mask[XIMaskLen(XI_LASTEVENT)] = {};
   XISetMask(mask, XI_HierarchyChanged);
+  XISetMask(mask, XI_KeyPress);
+  XISetMask(mask, XI_KeyRelease);
 
   evmask.deviceid = XIAllDevices;
   evmask.mask_len = sizeof(mask);
   evmask.mask = mask;
 
   Display* display = ui::GetXDisplay();
-  XISelectEvents(display, DefaultRootWindow(display), &evmask, 1);
+  XISelectEvents(display, ui::GetX11RootWindow(), &evmask, 1);
 }
 
 // Checks the |event| and asynchronously sets the XKB layout when necessary.
-void HandleHierarchyChangedEvent(XIHierarchyEvent* event) {
+void HandleHierarchyChangedEvent(
+    XIHierarchyEvent* event,
+    ObserverList<DeviceHierarchyObserver>* observer_list) {
   if (!(event->flags & XISlaveAdded))
     return;
 
   for (int i = 0; i < event->num_info; ++i) {
     XIHierarchyInfo* info = &event->info[i];
     if ((info->flags & XISlaveAdded) && (info->use == XIFloatingSlave)) {
+      FOR_EACH_OBSERVER(DeviceHierarchyObserver,
+                        *observer_list,
+                        DeviceAdded(info->deviceid));
+
       chromeos::input_method::InputMethodManager* input_method_manager =
           chromeos::input_method::InputMethodManager::GetInstance();
       chromeos::input_method::XKeyboard* xkeyboard =
@@ -63,8 +73,6 @@ void HandleHierarchyChangedEvent(XIHierarchyEvent* event) {
 
 }  // namespace
 
-namespace chromeos {
-
 // static
 XInputHierarchyChangedEventListener*
 XInputHierarchyChangedEventListener::GetInstance() {
@@ -108,16 +116,25 @@ bool XInputHierarchyChangedEventListener::ProcessedXEvent(XEvent* xevent) {
   }
 
   XGenericEventCookie* cookie = &(xevent->xcookie);
-  const bool hierarchy_changed = (cookie->evtype == XI_HierarchyChanged);
+  bool handled = false;
 
-  if (hierarchy_changed) {
+  if (cookie->evtype == XI_HierarchyChanged) {
     XIHierarchyEvent* event = static_cast<XIHierarchyEvent*>(cookie->data);
-    HandleHierarchyChangedEvent(event);
+    HandleHierarchyChangedEvent(event, &observer_list_);
     if (event->flags & XIDeviceEnabled || event->flags & XIDeviceDisabled)
       NotifyDeviceHierarchyChanged();
+    handled = true;
+  } else if (cookie->evtype == XI_KeyPress || cookie->evtype == XI_KeyRelease) {
+    XIDeviceEvent* xiev = reinterpret_cast<XIDeviceEvent*>(cookie->data);
+    if (xiev->deviceid == xiev->sourceid) {
+      FOR_EACH_OBSERVER(DeviceHierarchyObserver,
+                        observer_list_,
+                        KeyPressedOrReleased(xiev->deviceid));
+      handled = true;
+    }
   }
 
-  return hierarchy_changed;
+  return handled;
 }
 
 void XInputHierarchyChangedEventListener::NotifyDeviceHierarchyChanged() {