Fixes issue with accelerators when a menu is open

ash/accelerators/accelerator_dispatcher_linux.cc

+// 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.
+
+#include "ash/accelerators/accelerator_dispatcher.h"
+
+#include <X11/Xlib.h>
+
+#include "ash/accelerators/accelerator_controller.h"
+#include "ash/shell.h"
+#include "base/message_loop.h"
+#include "base/message_pump_x.h"
+#include "ui/aura/event.h"
+#include "ui/base/accelerators/accelerator.h"
+
+namespace ash {
+
+namespace {
+
+const int kModifierMask = (ui::EF_SHIFT_DOWN |
+                           ui::EF_CONTROL_DOWN |
+                           ui::EF_ALT_DOWN);
+}  // namespace
+
+base::MessagePumpDispatcher::DispatchStatus AcceleratorDispatcher::Dispatch(
+    XEvent* xev) {
+  ash::Shell* shell = ash::Shell::GetInstance();
+  if (shell->IsScreenLocked())
+    return EVENT_PROCESSED;

[oshima, 2012/01/31 18:19:52]

I believe you need to call

return RootWindow::GetDispatcher()->Dispatch(event);

otherwise it will drop all events.

+
+  if (xev->type == KeyPress) {
+    ash::AcceleratorController* accelerator_controller =
+        shell->accelerator_controller();
+    ui::Accelerator accelerator(ui::KeyboardCodeFromNative(xev),
+                                ui::EventFlagsFromNative(xev) & kModifierMask);
+    if (accelerator_controller && accelerator_controller->Process(accelerator))
+      return EVENT_PROCESSED;
+  }
+  return nested_dispatcher_->Dispatch(xev);
+}
+
+}  // namespace ash