* gtkutil.c (xg_event_is_for_menubar): Also check that event window

is related to the menu bar (Bug#6499).

diff --git a/src/ChangeLog b/src/ChangeLog
index c00d4febc2d2e2722c71d68bc88e4d0fcf5f9b2d..c3d77fc754dc8261ecae1c0c1d4e6b005bc5cca4 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,8 @@
+2010-07-17  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * gtkutil.c (xg_event_is_for_menubar): Also check that event window
+       is related to the menu bar (Bug#6499).
+
 2010-07-14  Jan Djärv  <jan.h.d@swipnet.se>
 
        * xterm.h (x_menubar_window_to_frame): Second parameter is XEvent*.

diff --git a/src/gtkutil.c b/src/gtkutil.c
index 7c3484fbeb50d8ff638c6e31b9e7a70c3ca06b0e..1a90aabc1c63b183818545dad0fd7baa3ee7a145 100644 (file)
--- a/src/gtkutil.c
+++ b/src/gtkutil.c
@@ -2994,6 +2994,13 @@ int
 xg_event_is_for_menubar (FRAME_PTR f, XEvent *event)
 {
   struct x_output *x = f->output_data.x;
+  GList *iter;
+  GdkRectangle rec;
+  GList *list;
+  GdkDisplay *gdpy;
+  GdkWindow *gw;
+  GdkEvent gevent;
+  GtkWidget *gwdesc;
 
   if (! x->menubar_widget) return 0;
 
@@ -3004,14 +3011,24 @@ xg_event_is_for_menubar (FRAME_PTR f, XEvent *event)
          && event->xbutton.same_screen))
     return 0;
 
-  GList *list = gtk_container_get_children (GTK_CONTAINER (x->menubar_widget));
+  gdpy = gdk_x11_lookup_xdisplay (FRAME_X_DISPLAY (f));
+  gw = gdk_xid_table_lookup_for_display (gdpy, event->xbutton.window);
+  if (! gw) return 0;
+  gevent.any.window = gw;
+  gwdesc = gtk_get_event_widget (&gevent);
+  if (! gwdesc) return 0;
+  if (! GTK_IS_MENU_BAR (gwdesc)
+      && ! GTK_IS_MENU_ITEM (gwdesc)
+      && ! gtk_widget_is_ancestor (x->menubar_widget, gwdesc))
+    return 0;
+
+  list = gtk_container_get_children (GTK_CONTAINER (x->menubar_widget));
   if (! list) return 0;
-  GList *iter;
-  GdkRectangle rec;
   rec.x = event->xbutton.x;
   rec.y = event->xbutton.y;
   rec.width = 1;
   rec.height = 1;
+
   for (iter = list ; iter; iter = g_list_next (iter))
     {
       GtkWidget *w = GTK_WIDGET (iter->data);