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_AURA) && defined(USE_X11)
+#include "ui/aura/root_window.h"
+#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 742 matching lines @@
   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
 
+#if defined(USE_AURA) && defined(USE_X11)
+    aura::RootWindow* root_window =
+        source->GetWidget()->GetNativeWindow()->GetRootWindow();
+#endif
+
+    gfx::Point screen_loc(event.location());
+    View::ConvertPointToScreen(source->GetScrollViewContainer(), &screen_loc);
+
     // 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;
     }
     Cancel(exit_type);
+
+#if defined(USE_AURA) && defined(USE_X11)
+    if (exit_type == EXIT_ALL && event.type() == ui::ET_MOUSE_PRESSED) {
+      XEvent xevent = {0};
+      xevent.xbutton.x = screen_loc.x();
+      xevent.xbutton.y = screen_loc.y();
+      xevent.xbutton.type = ButtonPress;
+      xevent.xbutton.same_screen = True;
+      const int event_flags = event.flags();
+      bool do_it = true;
+      if (event_flags & ui::EF_LEFT_MOUSE_BUTTON) {
+        xevent.xbutton.button = Button1;
+        xevent.xbutton.state = Button1Mask;
+      } else if (event_flags & ui::EF_MIDDLE_MOUSE_BUTTON) {
+        xevent.xbutton.button = Button2;
+        xevent.xbutton.state = Button2Mask;
+      } else if (event_flags & ui::EF_RIGHT_MOUSE_BUTTON) {
+        xevent.xbutton.button = Button3;
+        xevent.xbutton.state = Button3Mask;
+      } else {
+        do_it = false;
+      }
+      // RootWindow will take care of other necessary fields.
+      if (do_it)
+        root_window->PostNativeEvent(&xevent);

[sky, 2013/01/07 16:06:13]

Is PostNativeEvent async? I think this code will roughly need to mirror the
existing RepostEvent. In fact this should be in RepostEvent for X11.

It's also worth talking with Oshima about this since he initially thought we
didn't need this logic (the comment you're removing on 799).

[sschmitz, 2013/01/08 22:48:57]

Done.

+    }
+#endif
+
     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