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

Side by Side Diff: chrome/browser/ui/views/extensions/extension_keybinding_registry_views.cc

Issue 10383240: This adds a webui overlay on the extensions page for showing what Extension keybindings are active.… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 7 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) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "chrome/browser/ui/views/extensions/extension_keybinding_registry_views .h" 5 #include "chrome/browser/ui/views/extensions/extension_keybinding_registry_views .h"
6 6
7 #include "chrome/browser/extensions/api/commands/extension_command_service.h" 7 #include "chrome/browser/extensions/api/commands/extension_command_service.h"
8 #include "chrome/browser/extensions/api/commands/extension_command_service_facto ry.h" 8 #include "chrome/browser/extensions/api/commands/extension_command_service_facto ry.h"
9 #include "chrome/browser/extensions/extension_browser_event_router.h" 9 #include "chrome/browser/extensions/extension_browser_event_router.h"
10 #include "chrome/browser/extensions/extension_service.h" 10 #include "chrome/browser/extensions/extension_service.h"
(...skipping 15 matching lines...) Expand all
26 focus_manager_->UnregisterAccelerator(iter->first, this); 26 focus_manager_->UnregisterAccelerator(iter->first, this);
27 } 27 }
28 28
29 void ExtensionKeybindingRegistryViews::AddExtensionKeybinding( 29 void ExtensionKeybindingRegistryViews::AddExtensionKeybinding(
30 const Extension* extension) { 30 const Extension* extension) {
31 ExtensionCommandService* command_service = 31 ExtensionCommandService* command_service =
32 ExtensionCommandServiceFactory::GetForProfile(profile_); 32 ExtensionCommandServiceFactory::GetForProfile(profile_);
33 // Add all the active keybindings (except page actions and browser actions, 33 // Add all the active keybindings (except page actions and browser actions,
34 // which are handled elsewhere). 34 // which are handled elsewhere).
35 const extensions::CommandMap& commands = 35 const extensions::CommandMap& commands =
36 command_service->GetActiveNamedCommands(extension->id()); 36 command_service->GetNamedCommands(extension->id(),
37 ExtensionCommandService::ACTIVE_ONLY);
37 extensions::CommandMap::const_iterator iter = commands.begin(); 38 extensions::CommandMap::const_iterator iter = commands.begin();
38 for (; iter != commands.end(); ++iter) { 39 for (; iter != commands.end(); ++iter) {
39 event_targets_[iter->second.accelerator()] = 40 event_targets_[iter->second.accelerator()] =
40 std::make_pair(extension->id(), iter->second.command_name()); 41 std::make_pair(extension->id(), iter->second.command_name());
41 focus_manager_->RegisterAccelerator( 42 focus_manager_->RegisterAccelerator(
42 iter->second.accelerator(), 43 iter->second.accelerator(),
43 ui::AcceleratorManager::kHighPriority, this); 44 ui::AcceleratorManager::kHighPriority, this);
44 } 45 }
45 } 46 }
46 47
(...skipping 25 matching lines...) Expand all
72 73
73 service->browser_event_router()->CommandExecuted( 74 service->browser_event_router()->CommandExecuted(
74 profile_, it->second.first, it->second.second); 75 profile_, it->second.first, it->second.second);
75 76
76 return true; 77 return true;
77 } 78 }
78 79
79 bool ExtensionKeybindingRegistryViews::CanHandleAccelerators() const { 80 bool ExtensionKeybindingRegistryViews::CanHandleAccelerators() const {
80 return true; 81 return true;
81 } 82 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698