Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(453)

Side by Side Diff: chrome/browser/chromeos/events/event_rewriter.cc

Issue 553243002: Track the active ExtensionKeybindingRegistry and make it available to EventRewriter. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lkcr
Patch Set: Address more comments. Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chromeos/events/event_rewriter.h" 5 #include "chrome/browser/chromeos/events/event_rewriter.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "ash/sticky_keys/sticky_keys_controller.h" 9 #include "ash/sticky_keys/sticky_keys_controller.h"
10 #include "ash/wm/window_state.h" 10 #include "ash/wm/window_state.h"
11 #include "ash/wm/window_util.h" 11 #include "ash/wm/window_util.h"
12 #include "base/command_line.h" 12 #include "base/command_line.h"
13 #include "base/logging.h" 13 #include "base/logging.h"
14 #include "base/macros.h" 14 #include "base/macros.h"
15 #include "base/prefs/pref_service.h" 15 #include "base/prefs/pref_service.h"
16 #include "base/strings/string_util.h" 16 #include "base/strings/string_util.h"
17 #include "base/sys_info.h" 17 #include "base/sys_info.h"
18 #include "chrome/browser/chromeos/login/ui/login_display_host_impl.h" 18 #include "chrome/browser/chromeos/login/ui/login_display_host_impl.h"
19 #include "chrome/browser/extensions/extension_commands_global_registry.h"
19 #include "chrome/browser/profiles/profile_manager.h" 20 #include "chrome/browser/profiles/profile_manager.h"
20 #include "chrome/common/pref_names.h" 21 #include "chrome/common/pref_names.h"
21 #include "chromeos/chromeos_switches.h" 22 #include "chromeos/chromeos_switches.h"
22 #include "chromeos/ime/ime_keyboard.h" 23 #include "chromeos/ime/ime_keyboard.h"
23 #include "chromeos/ime/input_method_manager.h" 24 #include "chromeos/ime/input_method_manager.h"
24 #include "components/user_manager/user_manager.h" 25 #include "components/user_manager/user_manager.h"
25 #include "ui/events/event.h" 26 #include "ui/events/event.h"
26 #include "ui/events/event_utils.h" 27 #include "ui/events/event_utils.h"
27 #include "ui/events/keycodes/keyboard_code_conversion.h" 28 #include "ui/events/keycodes/keyboard_code_conversion.h"
28 #include "ui/wm/core/window_util.h" 29 #include "ui/wm/core/window_util.h"
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
97 bool IsISOLevel5ShiftUsedByCurrentInputMethod() { 98 bool IsISOLevel5ShiftUsedByCurrentInputMethod() {
98 // Since both German Neo2 XKB layout and Caps Lock depend on Mod3Mask, 99 // Since both German Neo2 XKB layout and Caps Lock depend on Mod3Mask,
99 // it's not possible to make both features work. For now, we don't remap 100 // it's not possible to make both features work. For now, we don't remap
100 // Mod3Mask when Neo2 is in use. 101 // Mod3Mask when Neo2 is in use.
101 // TODO(yusukes): Remove the restriction. 102 // TODO(yusukes): Remove the restriction.
102 input_method::InputMethodManager* manager = 103 input_method::InputMethodManager* manager =
103 input_method::InputMethodManager::Get(); 104 input_method::InputMethodManager::Get();
104 return manager->IsISOLevel5ShiftUsedByCurrentInputMethod(); 105 return manager->IsISOLevel5ShiftUsedByCurrentInputMethod();
105 } 106 }
106 107
108 bool IsExtensionCommandRegistered(const ui::KeyEvent& key_event) {
109 // Some keyboard events for ChromeOS get rewritten, such as:
110 // Search+Shift+Left gets converted to Shift+Home (BeginDocument).
111 // This doesn't make sense if the user has assigned that shortcut
112 // to an extension. Because:
113 // 1) The extension would, upon seeing a request for Ctrl+Shift+Home have
114 // to register for Shift+Home, instead.
115 // 2) The conversion is unnecessary, because Shift+Home (BeginDocument) isn't
116 // going to be executed.
117 // Therefore, we skip converting the accelerator if an extension has
118 // registered for this shortcut.
119 Profile* profile = ProfileManager::GetActiveUserProfile();
120 if (!profile)
121 return false;
122
123 extensions::ExtensionKeybindingRegistry* registry =
124 extensions::ExtensionCommandsGlobalRegistry::Get(profile)
125 ->active_registry();
126 if (!registry)
127 return false;
128
129 int modifiers = ui::EF_NONE;
130 if (key_event.IsShiftDown())
131 modifiers |= ui::EF_SHIFT_DOWN;
132 if (key_event.IsControlDown())
133 modifiers |= ui::EF_CONTROL_DOWN;
134 if (key_event.IsAltDown())
135 modifiers |= ui::EF_ALT_DOWN;
136 if (key_event.IsCommandDown())
137 modifiers |= ui::EF_COMMAND_DOWN;
138 ui::Accelerator accelerator(key_event.key_code(), modifiers);
139 if (registry->IsAcceleratorRegistered(accelerator) ||
140 extensions::ExtensionCommandsGlobalRegistry::Get(profile)
141 ->IsAcceleratorRegistered(accelerator))
142 return true;
143 return false;
144 }
145
107 EventRewriter::DeviceType GetDeviceType(const std::string& device_name) { 146 EventRewriter::DeviceType GetDeviceType(const std::string& device_name) {
108 std::vector<std::string> tokens; 147 std::vector<std::string> tokens;
109 Tokenize(device_name, " .", &tokens); 148 Tokenize(device_name, " .", &tokens);
110 149
111 // If the |device_name| contains the two words, "apple" and "keyboard", treat 150 // If the |device_name| contains the two words, "apple" and "keyboard", treat
112 // it as an Apple keyboard. 151 // it as an Apple keyboard.
113 bool found_apple = false; 152 bool found_apple = false;
114 bool found_keyboard = false; 153 bool found_keyboard = false;
115 for (size_t i = 0; i < tokens.size(); ++i) { 154 for (size_t i = 0; i < tokens.size(); ++i) {
116 if (!found_apple && LowerCaseEqualsASCII(tokens[i], "apple")) 155 if (!found_apple && LowerCaseEqualsASCII(tokens[i], "apple"))
(...skipping 236 matching lines...) Expand 10 before | Expand all | Expand 10 after
353 remapped_state->key_code = map.output_key_code; 392 remapped_state->key_code = map.output_key_code;
354 remapped_state->flags = (input.flags & ~map.input_flags) | map.output_flags; 393 remapped_state->flags = (input.flags & ~map.input_flags) | map.output_flags;
355 return true; 394 return true;
356 } 395 }
357 return false; 396 return false;
358 } 397 }
359 398
360 ui::EventRewriteStatus EventRewriter::RewriteKeyEvent( 399 ui::EventRewriteStatus EventRewriter::RewriteKeyEvent(
361 const ui::KeyEvent& key_event, 400 const ui::KeyEvent& key_event,
362 scoped_ptr<ui::Event>* rewritten_event) { 401 scoped_ptr<ui::Event>* rewritten_event) {
402 if (IsExtensionCommandRegistered(key_event))
403 return ui::EVENT_REWRITE_CONTINUE;
363 if (key_event.source_device_id() != ui::ED_UNKNOWN_DEVICE) 404 if (key_event.source_device_id() != ui::ED_UNKNOWN_DEVICE)
364 DeviceKeyPressedOrReleased(key_event.source_device_id()); 405 DeviceKeyPressedOrReleased(key_event.source_device_id());
365 MutableKeyState state = {key_event.flags(), key_event.key_code()}; 406 MutableKeyState state = {key_event.flags(), key_event.key_code()};
366 // Do not rewrite an event sent by ui_controls::SendKeyPress(). See 407 // Do not rewrite an event sent by ui_controls::SendKeyPress(). See
367 // crbug.com/136465. 408 // crbug.com/136465.
368 if (!(key_event.flags() & ui::EF_FINAL)) { 409 if (!(key_event.flags() & ui::EF_FINAL)) {
369 RewriteModifierKeys(key_event, &state); 410 RewriteModifierKeys(key_event, &state);
370 RewriteNumPadKeys(key_event, &state); 411 RewriteNumPadKeys(key_event, &state);
371 } 412 }
372 ui::EventRewriteStatus status = ui::EVENT_REWRITE_CONTINUE; 413 ui::EventRewriteStatus status = ui::EVENT_REWRITE_CONTINUE;
(...skipping 488 matching lines...) Expand 10 before | Expand all | Expand 10 after
861 KeyboardDeviceAddedInternal(device_info[i].deviceid, device_info[i].name); 902 KeyboardDeviceAddedInternal(device_info[i].deviceid, device_info[i].name);
862 } 903 }
863 904
864 XIFreeDeviceInfo(device_info); 905 XIFreeDeviceInfo(device_info);
865 #else 906 #else
866 KeyboardDeviceAddedInternal(device_id, "keyboard"); 907 KeyboardDeviceAddedInternal(device_id, "keyboard");
867 #endif 908 #endif
868 } 909 }
869 910
870 } // namespace chromeos 911 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698