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

Unified Diff: ui/aura/root_window_host_x11.cc

Issue 67923002: Fixes for -Wunused-function on Linux, Android and ChromeOS (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase for realz Created 7 years, 1 month 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sandbox/linux/seccomp-bpf/sandbox_bpf.cc ('k') | ui/gfx/interpolated_transform_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/aura/root_window_host_x11.cc
diff --git a/ui/aura/root_window_host_x11.cc b/ui/aura/root_window_host_x11.cc
index af20138bea3f7be400bc2cd647b6144997091453..ec724acbf295e175035f11a2f23e2b700ea36303 100644
--- a/ui/aura/root_window_host_x11.cc
+++ b/ui/aura/root_window_host_x11.cc
@@ -126,46 +126,6 @@ void SelectEventsForRootWindow() {
#endif
}
-// We emulate Windows' WM_KEYDOWN and WM_CHAR messages. WM_CHAR events are only
-// generated for certain keys; see
-// http://msdn.microsoft.com/en-us/library/windows/desktop/ms646268.aspx. Per
-// discussion on http://crbug.com/108480, char events should furthermore not be
-// generated for Tab, Escape, and Backspace.
-bool ShouldSendCharEventForKeyboardCode(ui::KeyboardCode keycode) {
- if ((keycode >= ui::VKEY_0 && keycode <= ui::VKEY_9) ||
- (keycode >= ui::VKEY_A && keycode <= ui::VKEY_Z) ||
- (keycode >= ui::VKEY_NUMPAD0 && keycode <= ui::VKEY_NUMPAD9)) {
- return true;
- }
-
- switch (keycode) {
- case ui::VKEY_RETURN:
- case ui::VKEY_SPACE:
- // In addition to the keys listed at MSDN, we include other
- // graphic-character and numpad keys.
- case ui::VKEY_MULTIPLY:
- case ui::VKEY_ADD:
- case ui::VKEY_SUBTRACT:
- case ui::VKEY_DECIMAL:
- case ui::VKEY_DIVIDE:
- case ui::VKEY_OEM_1:
- case ui::VKEY_OEM_2:
- case ui::VKEY_OEM_3:
- case ui::VKEY_OEM_4:
- case ui::VKEY_OEM_5:
- case ui::VKEY_OEM_6:
- case ui::VKEY_OEM_7:
- case ui::VKEY_OEM_102:
- case ui::VKEY_OEM_PLUS:
- case ui::VKEY_OEM_COMMA:
- case ui::VKEY_OEM_MINUS:
- case ui::VKEY_OEM_PERIOD:
- return true;
- default:
- return false;
- }
-}
-
bool default_override_redirect = false;
} // namespace
« no previous file with comments | « sandbox/linux/seccomp-bpf/sandbox_bpf.cc ('k') | ui/gfx/interpolated_transform_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698