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

Unified Diff: chrome/browser/extensions/extension_keybinding_registry.h

Issue 838253004: MacViews: Fix duplicate definition of ExtensionKeyBindingRegistry::SetShortcutHandlingSuspended (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@DragBookmarks2
Patch Set: Created 5 years, 11 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/extension_keybinding_registry.h
diff --git a/chrome/browser/extensions/extension_keybinding_registry.h b/chrome/browser/extensions/extension_keybinding_registry.h
index 3936196e44d0272f2fa3f1ef687cae29ef6596bf..783ad6b5f825469342000f8800219631ae6224ef 100644
--- a/chrome/browser/extensions/extension_keybinding_registry.h
+++ b/chrome/browser/extensions/extension_keybinding_registry.h
@@ -57,9 +57,9 @@ class ExtensionKeybindingRegistry : public content::NotificationObserver,
~ExtensionKeybindingRegistry() override;
- // Enables/Disables general shortcut handling in Chrome. Implemented in
- // platform-specific ExtensionKeybindingsRegistry* files.
+ // Enables/Disables general shortcut handling in Chrome.
static void SetShortcutHandlingSuspended(bool suspended);
Devlin 2015/01/09 17:09:26 nit: Now that these are implemented here, please m
Andre 2015/01/10 00:48:10 The style guide is not specific about this, but my
+ static bool IsShortcutHandlingSuspended();
// Execute the command bound to |accelerator| and provided by the extension
// with |extension_id|, if it exists.
@@ -173,6 +173,13 @@ class ExtensionKeybindingRegistry : public content::NotificationObserver,
ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
extension_registry_observer_;
+ // Keeps track of whether shortcut handling is currently suspended. Shortcuts
+ // are suspended briefly while capturing which shortcut to assign to an
+ // extension command in the Config UI. If handling isn't suspended while
+ // capturing then trying to assign Ctrl+F to a command would instead result
+ // in the Find box opening.
+ static bool shortcut_handling_suspended_;
+
DISALLOW_COPY_AND_ASSIGN(ExtensionKeybindingRegistry);
};

Powered by Google App Engine
This is Rietveld 408576698