Polish the keybinding implementation a bit.

chrome/browser/ui/views/extensions/extension_keybinding_registry_views.cc

 // Copyright (c) 2012 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/ui/views/extensions/extension_keybinding_registry_views.h"
 
 #include "chrome/browser/extensions/extension_browser_event_router.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/extensions/extension.h"
@@ skipping 10 matching lines @@
 ExtensionKeybindingRegistryViews::~ExtensionKeybindingRegistryViews() {
   EventTargets::const_iterator iter;
   for (iter = event_targets_.begin(); iter != event_targets_.end(); ++iter)
     focus_manager_->UnregisterAccelerator(iter->first, this);
 }
 
 void ExtensionKeybindingRegistryViews::AddExtensionKeybinding(
     const Extension* extension) {
   // Add all the keybindings (except pageAction and browserAction, which are
   // handled elsewhere).
-  const std::vector<Extension::ExtensionKeybinding> commands =
-      extension->keybindings();
+  std::vector<Extension::ExtensionKeybinding> commands;
+  extension->GetCommandByType(Extension::ExtensionKeybinding::COMMANDS,
+                               &commands);
   for (size_t i = 0; i < commands.size(); ++i) {
-    if (ShouldIgnoreCommand(commands[i].command_name()))
-      continue;
-
     event_targets_[commands[i].accelerator()] =
         std::make_pair(extension->id(), commands[i].command_name());
     focus_manager_->RegisterAccelerator(
         commands[i].accelerator(), ui::AcceleratorManager::kHighPriority, this);
   }
 }
 
 void ExtensionKeybindingRegistryViews::RemoveExtensionKeybinding(
     const Extension* extension) {
   EventTargets::const_iterator iter;
@@ skipping 17 matching lines @@
 
   service->browser_event_router()->CommandExecuted(
       profile_, it->second.first, it->second.second);
 
   return true;
 }
 
 bool ExtensionKeybindingRegistryViews::CanHandleAccelerators() const {
   return true;
 }