Fixes issue with accelerators when a menu is open

ash/accelerators/accelerator_dispatcher.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 ASH_ACCELERATORS_ACCELERATOR_DISPATCHER_H_
+#define ASH_ACCELERATORS_ACCELERATOR_DISPATCHER_H_
+#pragma once
+
+#include "ash/ash_export.h"
+#include "base/message_loop.h"
+
+namespace ash {
+
+// Dispatcher for handling accelerators in ash.
+// Wraps a nested dispatcher to which control is passed if no accelerator key
+// has been pressed.
+
+#if defined(OS_MACOSX)
+class ASH_EXPORT AcceleratorDispatcher {
+#else
+class ASH_EXPORT AcceleratorDispatcher : public MessageLoop::Dispatcher {
+#endif  // defined(OS_MACOSX)
+ public:
+  explicit AcceleratorDispatcher(MessageLoop::Dispatcher* nested_dispatcher);
+
+#if defined(OS_POSIX)
+  virtual base::MessagePumpDispatcher::DispatchStatus Dispatch(
+      XEvent* xev) OVERRIDE;

[oshima, 2012/01/25 21:24:37]

This doesn't make sense for mac (OS_POSIX is defined for mac as well). I'd
recommend to change this to USE_X11 and remove OS_MACOSX for now.
Just Add TODO for mac port.

+#elif defined(OS_WIN)
+  bool AcceleratorDispatcher::Dispatch(const MSG& msg) OVERRIDE;
+#endif
+
+ private:
+  MessageLoop::Dispatcher* nested_dispatcher_;
+
+  DISALLOW_COPY_AND_ASSIGN(AcceleratorDispatcher);
+};
+
+}  // namespace ash
+
+#endif  // ASH_ACCELERATORS_ACCELERATOR_DISPATCHER_H_