Enable keyboard accelerators while a menu is open

ui/views/views_delegate.h

Index: ui/views/views_delegate.h
diff --git a/ui/views/views_delegate.h b/ui/views/views_delegate.h
index e2f33b05caa5e032b7575788c8e6b364d7b61e6d..0d2448f9a9d7094f495c291fe7f50c56144562d8 100644
--- a/ui/views/views_delegate.h
+++ b/ui/views/views_delegate.h
@@ -102,6 +102,10 @@ class VIEWS_EXPORT ViewsDelegate {
                                      int item_count,
                                      bool has_submenu);
 
+  // Invoked when an accelerator is pressed and a menu is shown. Intended as a
+  // way for accelerators to be processed while a menu is showing.
+  virtual void HandleKeyEventOnMenu(ui::KeyEvent* event);

[sky, 2015/09/09 15:44:52]

In looking at how you use this function I think it would be better to have the
following:

enum class ProcessAcceleratorResult {
  // The accelerator was handled and the menu should close.
  HANDLED,

  // The accelerator was not handled.
  NOT_HANDLED
};
virtual AcceleratorResult ProcessAcceleratorWhileMenuShowing(const
ui::Accelerator& accelerator);

[afakhry, 2015/09/09 20:21:33]

Done.

+
 #if defined(OS_WIN)
   // Retrieves the default window icon to use for windows if none is specified.
   virtual HICON GetDefaultWindowIcon() const;