[UI Views] Handle accelerators being pressed for buttons inside a menu

ui/views/controls/button/menu_button.h

Index: ui/views/controls/button/menu_button.h
diff --git a/ui/views/controls/button/menu_button.h b/ui/views/controls/button/menu_button.h
index d908db8b40d8e9b98b9171fdca72fe8ae0603c51..a98f41a51586dd2337fc6048e68b62d707989efa 100644
--- a/ui/views/controls/button/menu_button.h
+++ b/ui/views/controls/button/menu_button.h
@@ -78,6 +78,7 @@ class VIEWS_EXPORT MenuButton : public LabelButton {
   bool OnKeyPressed(const ui::KeyEvent& event) override;
   bool OnKeyReleased(const ui::KeyEvent& event) override;
   void GetAccessibleState(ui::AXViewState* state) override;
+  bool AcceleratorPressed(const ui::Accelerator& accelerator) override;

[varkha, 2016/02/06 00:58:08]

nit: should be above GetAccessibleState in view.h order.

[Devlin, 2016/02/06 02:04:23]

Huh.  I didn't know we had a style rule for matching inherited methods to the
order they are declared in the super class (only for matching declaration and
definition order).  I'm pretty sure most of our classes are quite wrong for that
style. :)

(But done - super easy change and no reason not to)

 
  protected:
   // Paint the menu marker image.