context menu does not work if one already open

ui/views/controls/menu/menu_controller.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 "ui/views/controls/menu/menu_controller.h"
 
 #include "base/i18n/case_conversion.h"
 #include "base/i18n/rtl.h"
 #include "base/run_loop.h"
 #include "base/time.h"
@@ skipping 16 matching lines @@
 #include "ui/views/view_constants.h"
 #include "ui/views/views_delegate.h"
 #include "ui/views/widget/root_view.h"
 #include "ui/views/widget/widget.h"
 
 #if defined(USE_AURA)
 #include "ui/aura/env.h"
 #include "ui/aura/window.h"
 #endif
 
+#if defined(USE_X11)
+#include <X11/Xlib.h>
+#endif
+
 using base::Time;
 using base::TimeDelta;
 using ui::OSExchangeData;
 
 // Period of the scroll timer (in milliseconds).
 static const int kScrollTimerMS = 30;
 
 // Amount of time from when the drop exits the menu and the menu is hidden.
 static const int kCloseOnExitTime = 1200;
 
@@ skipping 746 matching lines @@
   // is sent before the menu appears.  Ignore it.
   if ((event.flags() & ui::EF_RIGHT_MOUSE_BUTTON) &&
       (event.flags() & ui::EF_FROM_TOUCH))
     return;
 
   if (part.type == MenuPart::NONE ||
       (part.type == MenuPart::MENU_ITEM && part.menu &&
        part.menu->GetRootMenuItem() != state_.item->GetRootMenuItem())) {
     // Mouse wasn't pressed over any menu, or the active menu, cancel.
 
-    // We're going to close and we own the mouse capture. We need to repost the
-    // mouse down, otherwise the window the user clicked on won't get the
-    // event.
-#if defined(OS_WIN) && !defined(USE_AURA)
-    RepostEvent(source, event);
-    // NOTE: not reposting on linux seems fine.
-#endif
-
-    // And close.
     ExitType exit_type = EXIT_ALL;
     if (!menu_stack_.empty()) {
       // We're running nested menus. Only exit all if the mouse wasn't over one
       // of the menus from the last run.
       gfx::Point screen_loc(event.location());
       View::ConvertPointToScreen(source->GetScrollViewContainer(), &screen_loc);
       MenuPart last_part = GetMenuPartByScreenCoordinateUsingMenu(
           menu_stack_.back().item, screen_loc);
       if (last_part.type != MenuPart::NONE)
         exit_type = EXIT_OUTERMOST;
     }
+
+#if defined(OS_WIN) && !defined(USE_AURA)
+    // We're going to close and we own the mouse capture. We need to repost the
+    // mouse down, otherwise the window the user clicked on won't get the
+    // event.
+    RepostEvent(source, event);
+#elif defined(USE_X11)
+    if (exit_type == EXIT_ALL && state_.item && event.native_event()) {
+      // We will putback the X11 event here, so that the usual
+      // action (e.g. window activation, new context menu) happens
+      // after this context menu and its message loop have exited. We
+      // do not want mouse button events treated as double clicks.
+      // Without the following special precaution a "put back" mouse
+      // button event will result in a double click. We use the
+      // "send_event" flag. When this event is processed later on, the
+      // flag is used to avoid treating it as a double click.  The
+      // field "send_event" is in the same place for all event types
+      // so we can use "xany" regardless of event type.
+      XEvent xevent = *event.native_event();
+      xevent.xany.send_event = True;
+      XPutBackEvent(xevent.xany.display, &xevent);
+    }
+#endif
+
+    // And close.
     Cancel(exit_type);
+
     return;
   }
 
   // On a press we immediately commit the selection, that way a submenu
   // pops up immediately rather than after a delay.
   int selection_types = SELECTION_UPDATE_IMMEDIATELY;
   if (!part.menu) {
     part.menu = part.parent;
     selection_types |= SELECTION_OPEN_SUBMENU;
   } else {
@@ skipping 1311 matching lines @@
              (!pending_state_.item->HasSubmenu() ||
               !pending_state_.item->GetSubmenu()->IsShowing())) {
     // On exit if the user hasn't selected an item with a submenu, move the
     // selection back to the parent menu item.
     SetSelection(pending_state_.item->GetParentMenuItem(),
                  SELECTION_OPEN_SUBMENU);
   }
 }
 
 }  // namespace views