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

Unified Diff: chrome/browser/extensions/global_shortcut_listener_win.cc

Issue 23812010: Implement first part of supporting global extension commands. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Polishing a bit Created 7 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/global_shortcut_listener_win.cc
diff --git a/chrome/browser/extensions/global_shortcut_listener_win.cc b/chrome/browser/extensions/global_shortcut_listener_win.cc
new file mode 100644
index 0000000000000000000000000000000000000000..db5a973ba33238cb681ba23280ecaf676103f657
--- /dev/null
+++ b/chrome/browser/extensions/global_shortcut_listener_win.cc
@@ -0,0 +1,80 @@
+// Copyright (c) 2013 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/extensions/global_shortcut_listener_win.h"
+
+#include "base/win/win_util.h"
+#include "content/public/browser/browser_thread.h"
+#include "ui/base/accelerators/accelerator.h"
+#include "ui/base/events/event_constants.h"
+#include "ui/base/keycodes/keyboard_code_conversion_win.h"
zhchbin 2013/09/20 07:49:23 This file has been moved from base to events. s/ba
+
+using content::BrowserThread;
+
+namespace {
+
+static base::LazyInstance<extensions::GlobalShortcutListenerWin> instance =
+ LAZY_INSTANCE_INITIALIZER;
+
+} // namespace
+
+namespace extensions {
+
+// static
+GlobalShortcutListener* GlobalShortcutListener::GetInstance() {
+ CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+ return instance.Pointer();
+}
+
+GlobalShortcutListenerWin::GlobalShortcutListenerWin()
+ : keyboard_hook_(NULL),
+ is_hooking_(false) {
+ CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+}
+
+GlobalShortcutListenerWin::~GlobalShortcutListenerWin() {
+ if (is_hooking_)
+ StopHooking();
+}
+
+void GlobalShortcutListenerWin::StartHooking() {
+ if (is_hooking_)
+ return;
+
+ keyboard_hook_ = SetWindowsHookEx(WH_KEYBOARD_LL, &KeyboardHook, NULL, 0L);
zhchbin 2013/09/20 07:49:23 Do you mind explaining the reason that you chose u
Finnur 2013/09/20 13:01:12 I haven't fully made up my mind yet (and not fully
Finnur 2013/09/20 13:15:56 Well, I take that back: a window isn't required. B
zhchbin 2013/09/21 02:07:50 We can use base::win::MessageWindow to subscribe t
Finnur 2013/09/21 12:23:50 Yes, I came to the conclusion after responding to
+ if (keyboard_hook_)
+ is_hooking_ = true;
+}
+
+void GlobalShortcutListenerWin::StopHooking() {
+ if (!is_hooking_)
+ return;
+
+ if (keyboard_hook_)
+ UnhookWindowsHookEx(keyboard_hook_);
+ keyboard_hook_ = NULL;
+ is_hooking_ = false;
+}
+
+// static
+LRESULT CALLBACK GlobalShortcutListenerWin::KeyboardHook(int code,
+ WPARAM w_param,
+ LPARAM l_param) {
+ if (code >= 0 && w_param == WM_KEYDOWN) {
+ int vkCode = reinterpret_cast<KBDLLHOOKSTRUCT*>(l_param)->vkCode;
+ int modifiers = 0;
+ modifiers |= base::win::IsShiftPressed() ? ui::EF_SHIFT_DOWN : 0;
+ modifiers |= base::win::IsCtrlPressed() ? ui::EF_CONTROL_DOWN : 0;
+ modifiers |= base::win::IsAltPressed() ? ui::EF_ALT_DOWN : 0;
+ ui::Accelerator accelerator(
+ ui::KeyboardCodeForWindowsKeyCode(vkCode), modifiers);
+
+ if (instance.Get().NotifyKeyPressed(accelerator))
+ return -1; // Prevent further processing.
+ }
+
+ return CallNextHookEx(NULL, code, w_param, l_param);
+}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698