Apply Ash's key rewriter to an aura::MouseEvent so that Modifler+Click is handled correctly

chrome/browser/ui/views/ash/key_rewriter.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/ash/key_rewriter.h"
 
 #include <vector>
 
 #include "ash/shell.h"
 #include "base/logging.h"
@@ skipping 171 matching lines @@
   const ash::KeyRewriterDelegate::Action kActionRewrite =
       ash::KeyRewriterDelegate::ACTION_REWRITE_EVENT;
   if (!event->HasNativeEvent()) {
     // Do not handle a fabricated event generated by tests.
     return kActionRewrite;
   }
   Rewrite(event);
   return kActionRewrite;  // Do not drop the event.
 }
 
+ash::KeyRewriterDelegate::Action KeyRewriter::RewriteOrFilterLocatedEvent(
+    aura::LocatedEvent* event) {
+  const ash::KeyRewriterDelegate::Action kActionRewrite =
+      ash::KeyRewriterDelegate::ACTION_REWRITE_EVENT;
+  if (!event->HasNativeEvent()) {
+    // Do not handle a fabricated event generated by tests.
+    return kActionRewrite;
+  }
+  RewriteLocatedEvent(event);
+  return kActionRewrite;

[Daniel Erat, 2012/07/09 18:18:28]

nit: just simplify this to three lines?

... KeyRewriter::RewriteOrFilterLocatedEvent(...) {
  if (event->HasNativeEvent())
    RewriteLocatedEvent(event);
  return ash::KeyRewriterDelegate::ACTION_REWRITE_EVENT;
}

or did you mean to return a different value in the !HasNativeEvent() case?

[Yusuke Sato, 2012/07/09 20:36:50]

Done.

+}
+
 void KeyRewriter::OnKeyboardMappingChanged(const aura::RootWindow* root) {
 #if defined(OS_CHROMEOS)
   RefreshKeycodes();
 #endif
 }
 
 #if defined(OS_CHROMEOS)
 void KeyRewriter::DeviceAdded(int device_id) {
   DCHECK_NE(XIAllDevices, device_id);
   DCHECK_NE(XIAllMasterDevices, device_id);
@@ skipping 144 matching lines @@
       device_id_to_type_.find(last_device_id_);
   if (iter == device_id_to_type_.end()) {
     LOG(ERROR) << "Device ID " << last_device_id_ << " is unknown.";
     return false;
   }
 
   const DeviceType type = iter->second;
   return type == kDeviceAppleKeyboard;
 }
 
+void KeyRewriter::GetRemappedModifierMasks(
+    int original_flags,
+    unsigned int original_native_modifiers,
+    int* remapped_flags,
+    unsigned int* remapped_native_modifiers) const {
+#if defined(OS_CHROMEOS)
+  // TODO(glotov): remove the following condition when we do not restart chrome
+  // when user logs in as guest. See Rewrite() for details.
+  if (chromeos::UserManager::Get()->IsLoggedInAsGuest() &&
+      chromeos::BaseLoginDisplayHost::default_host()) {
+    return;
+  }
+
+  const PrefService* pref_service =
+      pref_service_ ? pref_service_ : GetPrefService();
+  if (!pref_service)
+    return;
+
+  for (size_t i = 0; i < arraysize(kModifierFlagToPrefName); ++i) {
+    if (original_native_modifiers &
+        kModifierFlagToPrefName[i].native_modifier) {
+      const ModifierRemapping* remapped_key =
+          GetRemappedKey(kModifierFlagToPrefName[i].pref_name, *pref_service);
+      // Rewrite Command-L/R key presses on an Apple keyboard to Control-L/R.
+      if (IsAppleKeyboard() &&
+          (kModifierFlagToPrefName[i].native_modifier == Mod4Mask)) {
+        remapped_key = kModifierRemappingCtrl;
+      }
+      if (remapped_key) {
+        *remapped_flags |= remapped_key->flag;
+        *remapped_native_modifiers |= remapped_key->native_modifier;
+      } else {
+        *remapped_flags |= kModifierFlagToPrefName[i].flag;
+        *remapped_native_modifiers |=
+            kModifierFlagToPrefName[i].native_modifier;
+      }
+    }
+  }
+
+  *remapped_flags =
+      (original_flags & ~(ui::EF_CONTROL_DOWN | ui::EF_ALT_DOWN)) |
+      *remapped_flags;
+  *remapped_native_modifiers =
+      (original_native_modifiers & ~(Mod4Mask | ControlMask | Mod1Mask)) |
+      *remapped_native_modifiers;
+#endif
+}
+
 bool KeyRewriter::RewriteModifiers(aura::KeyEvent* event) {
   // Do nothing if we have just logged in as guest but have not restarted chrome
   // process yet (so we are still on the login screen). In this situations we
   // have no user profile so can not do anything useful.
   // Note that currently, unlike other accounts, when user logs in as guest, we
   // restart chrome process. In future this is to be changed.
   // TODO(glotov): remove the following condition when we do not restart chrome
   // when user logs in as guest.
  #if defined(OS_CHROMEOS)
    if (chromeos::UserManager::Get()->IsLoggedInAsGuest() &&
        chromeos::BaseLoginDisplayHost::default_host())
      return false;
  #endif  // defined(OS_CHROMEOS)
   const PrefService* pref_service =
       pref_service_ ? pref_service_ : GetPrefService();
   if (!pref_service)
     return false;
 
 #if defined(OS_CHROMEOS)
   DCHECK_EQ(chromeos::input_method::kControlKey,
             kModifierRemappingCtrl->remap_to);
 
   XEvent* xev = event->native_event();
   XKeyEvent* xkey = &(xev->xkey);
   KeySym keysym = XLookupKeysym(xkey, 0);
 
   ui::KeyboardCode remapped_keycode = event->key_code();
   KeyCode remapped_native_keycode = xkey->keycode;
-  int remapped_flags = 0;
-  unsigned int remapped_native_modifiers = 0U;
 
   // First, remap |keysym|.
   const char* pref_name = NULL;
   switch (keysym) {
     case XK_Super_L:
     case XK_Super_R:
       pref_name = prefs::kLanguageXkbRemapSearchKeyTo;
       break;
     case XK_Control_L:
     case XK_Control_R:
@@ skipping 17 matching lines @@
     if (remapped_key) {
       remapped_keycode = remapped_key->keycode;
       const size_t level = (event->IsShiftDown() ? (1 << 1) : 0) +
           (IsRight(keysym) ? (1 << 0) : 0);
       const KeySym native_keysym = remapped_key->native_keysyms[level];
       remapped_native_keycode = NativeKeySymToNativeKeycode(native_keysym);
     }
   }
 
   // Next, remap modifier bits.
-  for (size_t i = 0; i < arraysize(kModifierFlagToPrefName); ++i) {
-    if (xkey->state & kModifierFlagToPrefName[i].native_modifier) {
-      const ModifierRemapping* remapped_key =
-          GetRemappedKey(kModifierFlagToPrefName[i].pref_name, *pref_service);
-      // Rewrite Command-L/R key presses on an Apple keyboard to Control-L/R.
-      if (IsAppleKeyboard() &&
-          (kModifierFlagToPrefName[i].native_modifier == Mod4Mask)) {
-        remapped_key = kModifierRemappingCtrl;
-      }
-      if (remapped_key) {
-        remapped_flags |= remapped_key->flag;
-        remapped_native_modifiers |= remapped_key->native_modifier;
-      } else {
-        remapped_flags |= kModifierFlagToPrefName[i].flag;
-        remapped_native_modifiers |= kModifierFlagToPrefName[i].native_modifier;
-      }
-    }
-  }
-
-  remapped_flags = (event->flags() & ~(ui::EF_CONTROL_DOWN | ui::EF_ALT_DOWN)) |
-      remapped_flags;
-  remapped_native_modifiers =
-      (xkey->state & ~(Mod4Mask | ControlMask | Mod1Mask)) |
-      remapped_native_modifiers;
+  int remapped_flags = 0;
+  unsigned int remapped_native_modifiers = 0U;
+  GetRemappedModifierMasks(event->flags(), xkey->state,
+                           &remapped_flags, &remapped_native_modifiers);
 
   // Toggle Caps Lock if the remapped key is ui::VKEY_CAPITAL, but do nothing if
   // the original key is ui::VKEY_CAPITAL (i.e. a Caps Lock key on an external
   // keyboard is pressed) since X can handle that case.
   if ((event->type() == ui::ET_KEY_PRESSED) &&
       (event->key_code() != ui::VKEY_CAPITAL) &&
       (remapped_keycode == ui::VKEY_CAPITAL)) {
     chromeos::input_method::XKeyboard* xkeyboard = xkeyboard_ ?
         xkeyboard_ : InputMethodManager::GetInstance()->GetXKeyboard();
     xkeyboard->SetCapsLockEnabled(!xkeyboard->CapsLockIsEnabled());
@@ skipping 131 matching lines @@
     OverwriteEvent(event, next_xkeycode_, xkey->state & ~Mod1Mask,
                    ui::VKEY_NEXT, event->flags() & ~ui::EF_ALT_DOWN);
     rewritten = true;
   }
 #else
   // TODO(yusukes): Support Ash on other platforms if needed.
 #endif
   return rewritten;
 }
 
+void KeyRewriter::RewriteLocatedEvent(aura::LocatedEvent* event) {
+#if defined(OS_CHROMEOS)
+  XEvent* xevent = event->native_event();
+  if (!xevent || xevent->type != GenericEvent)
+    return;
+
+  XIDeviceEvent* xievent = static_cast<XIDeviceEvent*>(xevent->xcookie.data);
+  if (xievent->evtype != XI_ButtonPress && xievent->evtype != XI_ButtonRelease)
+    return;
+
+  // First, remap modifier masks.
+  int remapped_flags = 0;
+  unsigned int remapped_native_modifiers = 0U;
+  GetRemappedModifierMasks(event->flags(), xievent->mods.effective,
+                           &remapped_flags, &remapped_native_modifiers);
+  xievent->mods.effective = remapped_native_modifiers;
+
+  // Then, remap Alt+Button1 to Button3.
+  if ((xievent->mods.effective & Mod1Mask) && xievent->detail == 1) {
+    xievent->mods.effective &= ~Mod1Mask;
+    xievent->detail = 3;
+    if (xievent->evtype == XI_ButtonRelease) {
+      // On the release clear the left button from the existing state and the
+      // mods, and set the right button.
+      XISetMask(xievent->buttons.mask, 3);
+      XIClearMask(xievent->buttons.mask, 1);
+      xievent->mods.effective &= ~Button1Mask;
+    }
+  }
+
+  event->set_flags(ui::EventFlagsFromNative(xevent));
+#else
+  // TODO(yusukes): Support Ash on other platforms if needed.
+#endif
+}
+
 void KeyRewriter::OverwriteEvent(aura::KeyEvent* event,
                                  unsigned int new_native_keycode,
                                  unsigned int new_native_state,
                                  ui::KeyboardCode new_keycode,
                                  int new_flags) {
 #if defined(OS_CHROMEOS)
   XEvent* xev = event->native_event();
   XKeyEvent* xkey = &(xev->xkey);
   xkey->keycode = new_native_keycode;
   xkey->state = new_native_state;
@@ skipping 12 matching lines @@
   const DeviceType type = KeyRewriter::GetDeviceType(device_name);
   if (type == kDeviceAppleKeyboard) {
     VLOG(1) << "Apple keyboard '" << device_name << "' connected: "
             << "id=" << device_id;
   }
   // Always overwrite the existing device_id since the X server may reuse a
   // device id for an unattached device.
   device_id_to_type_[device_id] = type;
   return type;
 }