Fixes issue with accelerators when a menu is open

ash/accelerators/accelerator_handler.h

+// Copyright (c) 2011 The Chromium Authors. All rights reserved.

[oshima, 2012/01/23 18:06:39]

2012

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef ASH_ACCELERATORS_ACCELERATOR_HANDLER_H_
+#define ASH_ACCELERATORS_ACCELERATOR_HANDLER_H_
+#pragma once
+
+#include "ash/ash_export.h"
+#include "base/message_loop.h"
+
+namespace ash {
+
+#if defined(OS_MACOSX)
+class ASH_EXPORT AcceleratorHandler {
+#else
+class ASH_EXPORT AcceleratorHandler : public MessageLoop::Dispatcher {

[oshima, 2012/01/23 18:06:39]

There is an "AcceleratorHandler" class in views/focus/accelerator_handler.h, so
it'd be better to use different name.

[pkotwicz, 2012/01/23 19:33:02]

Oshima, do you have any suggestions for names?

[oshima, 2012/01/23 21:40:52]

AcceleratorDispatcher? Hmm, I'm not good at coming up good name. Scott, WDYT?

+#endif //defined(OS_MACOSX)
+ public:
+  AcceleratorHandler(MessageLoop::Dispatcher* nested_dispatcher);

[oshima, 2012/01/23 18:06:39]

explicit

+
+#if defined(OS_LINUX)
+  virtual base::MessagePumpDispatcher::DispatchStatus Dispatch(
+      XEvent* xev) OVERRIDE;
+#endif
+ protected:

[oshima, 2012/01/23 18:06:39]

can this be private?

+  MessageLoop::Dispatcher* nested_dispatcher_;

[oshima, 2012/01/23 18:06:39]

DISALLOW_COPY_AND_ASSIGN

+};
+
+} // namespace ash
+
+#endif // ASH_ACCELERATORS_ACCELERATOR_HANDLER_H_