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

Unified Diff: ui/views/controls/button/menu_button.h

Issue 1671323002: [UI Views] Handle accelerators being pressed for buttons inside a menu (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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 side-by-side diff with in-line comments
Download patch
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
Devlin 2016/02/06 02:04:23 Huh. I didn't know we had a style rule for matchi
protected:
// Paint the menu marker image.

Powered by Google App Engine
This is Rietveld 408576698