events: Remove ability to terminate a nested message-loop from the event-source.

ash/accelerators/accelerator_dispatcher_linux.cc

 // Copyright 2014 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 "base/memory/scoped_ptr.h"
 #include "base/run_loop.h"
 #include "ui/events/event.h"
 #include "ui/events/platform/platform_event_dispatcher.h"
@@ skipping 51 matching lines @@
 
   virtual uint32_t DispatchEvent(const ui::PlatformEvent& event) OVERRIDE {
     if (IsKeyEvent(event)) {
       ui::KeyEvent key_event(event, false);
       if (MenuClosedForPossibleAccelerator(key_event)) {
 #if defined(USE_X11)
         XPutBackEvent(event->xany.display, event);
 #else
         NOTIMPLEMENTED();
 #endif
-        return ui::POST_DISPATCH_QUIT_LOOP;
+        return ui::POST_DISPATCH_NONE;

[sadrul, 2014/05/14 17:46:33]

MenuClosedForPossibleAccelerator() (line 66) calls MenuController::CancelAll(),
which in turn calls MenuController::SetExitType(), which should terminate the
nested message-loop when appropriate. So it should not be necessary for the
event-source to terminate it again.

       }
 
       if (AcceleratorProcessedForKeyEvent(key_event))
         return ui::POST_DISPATCH_NONE;
     }
 
     ui::PlatformEventDispatcher* prev = *restore_dispatcher_;
     return prev ? prev->DispatchEvent(event)
                 : ui::POST_DISPATCH_PERFORM_DEFAULT;
   }
 
   scoped_ptr<ui::ScopedEventDispatcher> restore_dispatcher_;
 
   DISALLOW_COPY_AND_ASSIGN(AcceleratorDispatcherLinux);
 };
 
 scoped_ptr<AcceleratorDispatcher> AcceleratorDispatcher::Create(
     base::MessagePumpDispatcher* nested_dispatcher) {
   return scoped_ptr<AcceleratorDispatcher>(new AcceleratorDispatcherLinux());
 }
 
 }  // namespace ash