Track the active ExtensionKeybindingRegistry and make it available to EventRewriter.

chrome/browser/extensions/extension_commands_global_registry.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_COMMANDS_GLOBAL_REGISTRY_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_COMMANDS_GLOBAL_REGISTRY_H_
 
 #include <map>
 #include <string>
 
@@ skipping 15 matching lines @@
 // API).
 // Note: It handles regular extension commands (not browserAction and pageAction
 // popups, which are not bindable to global shortcuts). This class registers the
 // accelerators on behalf of the extensions and routes the commands to them via
 // the BrowserEventRouter.
 class ExtensionCommandsGlobalRegistry
     : public BrowserContextKeyedAPI,
       public ExtensionKeybindingRegistry,
       public GlobalShortcutListener::Observer {
  public:
+  ExtensionKeybindingRegistry* active_registry() { return active_registry_; }
+
+  void set_active_registry(ExtensionKeybindingRegistry* registry) {
+    active_registry_ = registry;
+  }
+
   // BrowserContextKeyedAPI implementation.
   static BrowserContextKeyedAPIFactory<ExtensionCommandsGlobalRegistry>*
       GetFactoryInstance();
 
   // Convenience method to get the ExtensionCommandsGlobalRegistry for a
   // profile.
   static ExtensionCommandsGlobalRegistry* Get(content::BrowserContext* context);
 
   // Enables/Disables global shortcut handling in Chrome.
   static void SetShortcutHandlingSuspended(bool suspended);
@@ skipping 17 matching lines @@
       const ui::Accelerator& accelerator,
       const std::string& command_name) OVERRIDE;
 
   // Called by the GlobalShortcutListener object when a shortcut this class has
   // registered for has been pressed.
   virtual void OnKeyPressed(const ui::Accelerator& accelerator) OVERRIDE;
 
   // Weak pointer to our browser context. Not owned by us.
   content::BrowserContext* browser_context_;
 
+  // The active ExtensionKeybindingRegistry.

[Devlin, 2014/09/12 00:10:33]

I might be a bit more specific here and say something like "The
ExtensionKeybindingRegistry that belongs to the active window. Only valid if
using TOOLKIT_VIEWS."

Whether or not we should #ifdef this code out for non-toolkit code... I don't
mind it being here (since it will be NULL otherwise, which is a logical value),
but I'll leave it to Finnur.

[David Tseng, 2014/09/12 01:26:58]

Done (the comment part).

[Finnur, 2014/09/12 11:25:44]

Rather than if-def'ing I'm actually more interested in getting this to work on
other platforms, as I think this is useful going forward. But that's probably
beyond the scope of this.

+  ExtensionKeybindingRegistry* active_registry_;
+
   DISALLOW_COPY_AND_ASSIGN(ExtensionCommandsGlobalRegistry);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_COMMANDS_GLOBAL_REGISTRY_H_