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

Side by Side Diff: chrome/browser/extensions/extension_commands_global_registry.h

Issue 178193030: Rename ProfileKeyedAPI to BrowserContextKeyedAPI and GetProfile to Get. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: nits Created 6 years, 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_COMMANDS_GLOBAL_REGISTRY_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_COMMANDS_GLOBAL_REGISTRY_H_
6 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_COMMANDS_GLOBAL_REGISTRY_H_ 6 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_COMMANDS_GLOBAL_REGISTRY_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 10
11 #include "base/compiler_specific.h" 11 #include "base/compiler_specific.h"
12 #include "chrome/browser/extensions/api/profile_keyed_api_factory.h"
13 #include "chrome/browser/extensions/extension_keybinding_registry.h" 12 #include "chrome/browser/extensions/extension_keybinding_registry.h"
14 #include "chrome/browser/extensions/global_shortcut_listener.h" 13 #include "chrome/browser/extensions/global_shortcut_listener.h"
14 #include "extensions/browser/browser_context_keyed_api_factory.h"
15 #include "ui/base/accelerators/accelerator.h" 15 #include "ui/base/accelerators/accelerator.h"
16 16
17 namespace content { 17 namespace content {
18 class BrowserContext; 18 class BrowserContext;
19 } 19 }
20 20
21 namespace extensions { 21 namespace extensions {
22 class Extension; 22 class Extension;
23 23
24 // ExtensionCommandsGlobalRegistry is a class that handles the cross-platform 24 // ExtensionCommandsGlobalRegistry is a class that handles the cross-platform
25 // implementation of the global shortcut registration for the Extension Commands 25 // implementation of the global shortcut registration for the Extension Commands
26 // API). 26 // API).
27 // Note: It handles regular extension commands (not browserAction and pageAction 27 // Note: It handles regular extension commands (not browserAction and pageAction
28 // popups, which are not bindable to global shortcuts). This class registers the 28 // popups, which are not bindable to global shortcuts). This class registers the
29 // accelerators on behalf of the extensions and routes the commands to them via 29 // accelerators on behalf of the extensions and routes the commands to them via
30 // the BrowserEventRouter. 30 // the BrowserEventRouter.
31 class ExtensionCommandsGlobalRegistry 31 class ExtensionCommandsGlobalRegistry
32 : public ProfileKeyedAPI, 32 : public BrowserContextKeyedAPI,
33 public ExtensionKeybindingRegistry, 33 public ExtensionKeybindingRegistry,
34 public GlobalShortcutListener::Observer { 34 public GlobalShortcutListener::Observer {
35 public: 35 public:
36 // ProfileKeyedAPI implementation. 36 // BrowserContextKeyedAPI implementation.
37 static ProfileKeyedAPIFactory< 37 static BrowserContextKeyedAPIFactory<ExtensionCommandsGlobalRegistry>*
38 ExtensionCommandsGlobalRegistry>* GetFactoryInstance(); 38 GetFactoryInstance();
39 39
40 // Convenience method to get the ExtensionCommandsGlobalRegistry for a 40 // Convenience method to get the ExtensionCommandsGlobalRegistry for a
41 // profile. 41 // profile.
42 static ExtensionCommandsGlobalRegistry* Get(content::BrowserContext* context); 42 static ExtensionCommandsGlobalRegistry* Get(content::BrowserContext* context);
43 43
44 explicit ExtensionCommandsGlobalRegistry(content::BrowserContext* context); 44 explicit ExtensionCommandsGlobalRegistry(content::BrowserContext* context);
45 virtual ~ExtensionCommandsGlobalRegistry(); 45 virtual ~ExtensionCommandsGlobalRegistry();
46 46
47 private: 47 private:
48 friend class ProfileKeyedAPIFactory<ExtensionCommandsGlobalRegistry>; 48 friend class BrowserContextKeyedAPIFactory<ExtensionCommandsGlobalRegistry>;
49 49
50 // ProfileKeyedAPI implementation. 50 // BrowserContextKeyedAPI implementation.
51 static const char* service_name() { 51 static const char* service_name() {
52 return "ExtensionCommandsGlobalRegistry"; 52 return "ExtensionCommandsGlobalRegistry";
53 } 53 }
54 54
55 // Overridden from ExtensionKeybindingRegistry: 55 // Overridden from ExtensionKeybindingRegistry:
56 virtual void AddExtensionKeybinding( 56 virtual void AddExtensionKeybinding(
57 const Extension* extension, 57 const Extension* extension,
58 const std::string& command_name) OVERRIDE; 58 const std::string& command_name) OVERRIDE;
59 virtual void RemoveExtensionKeybindingImpl( 59 virtual void RemoveExtensionKeybindingImpl(
60 const ui::Accelerator& accelerator, 60 const ui::Accelerator& accelerator,
61 const std::string& command_name) OVERRIDE; 61 const std::string& command_name) OVERRIDE;
62 62
63 // Called by the GlobalShortcutListener object when a shortcut this class has 63 // Called by the GlobalShortcutListener object when a shortcut this class has
64 // registered for has been pressed. 64 // registered for has been pressed.
65 virtual void OnKeyPressed(const ui::Accelerator& accelerator) OVERRIDE; 65 virtual void OnKeyPressed(const ui::Accelerator& accelerator) OVERRIDE;
66 66
67 // Weak pointer to our browser context. Not owned by us. 67 // Weak pointer to our browser context. Not owned by us.
68 content::BrowserContext* browser_context_; 68 content::BrowserContext* browser_context_;
69 69
70 DISALLOW_COPY_AND_ASSIGN(ExtensionCommandsGlobalRegistry); 70 DISALLOW_COPY_AND_ASSIGN(ExtensionCommandsGlobalRegistry);
71 }; 71 };
72 72
73 } // namespace extensions 73 } // namespace extensions
74 74
75 #endif // CHROME_BROWSER_EXTENSIONS_EXTENSION_COMMANDS_GLOBAL_REGISTRY_H_ 75 #endif // CHROME_BROWSER_EXTENSIONS_EXTENSION_COMMANDS_GLOBAL_REGISTRY_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698