Implement Keybinding for script badges.

chrome/browser/ui/views/location_bar/page_action_image_view.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_PAGE_ACTION_IMAGE_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_PAGE_ACTION_IMAGE_VIEW_H_
 
 #include <map>
 #include <string>
 
@@ skipping 111 matching lines @@
 
   // This is used for post-install visual feedback. The page_action icon is
   // briefly shown even if it hasn't been enabled by its extension.
   bool preview_enabled_;
 
   // The current popup and the button it came from.  NULL if no popup.
   ExtensionPopup* popup_;
 
   content::NotificationRegistrar registrar_;
 
-  // The extension keybinding accelerator this page action is listening for (to
+  // The extension command accelerator this page action is listening for (to
   // show the popup).
-  scoped_ptr<ui::Accelerator> keybinding_;
+  scoped_ptr<ui::Accelerator> keybinding_page_action_;

[Yoyo Zhou, 2012/08/02 09:59:29]

How about page_action_keybinding_? Seems to fit more naturally.

+  // The extension command accelerator this script badge is listening for (to
+  // show the popup).
+  scoped_ptr<ui::Accelerator> keybinding_script_badge_;
 
   scoped_ptr<views::MenuRunner> menu_runner_;
 
   // Fade-in animation for the icon with observer scoped to this.
   ExtensionAction::IconAnimation::ScopedObserver
       scoped_icon_animation_observer_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(PageActionImageView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_PAGE_ACTION_IMAGE_VIEW_H_