callisto-debian/gtk3
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

gdkwindow: subsurface in gdk_window_get_effective_parent()

Browse Source 
When the GtkWidget hierarchy does not match the GdkWindow hierarchy, the
GtkWidget code may find a common ancestor that cannot be found while
traversing the GdkWindow tree using gdk_window_get_effective_parent().

This happens with for example on Wayland, a GtkPopover has another
GtkPopover as parent, in this case, the GdkWindow parent is the root
window, whereas the GtkWidget parent is the other GtkPopover.

That confuses the gtk_widget_translate_coordinates() logic which will
bail out in this case and won't return the translated coordinates.

Make gdk_window_get_effective_parent() aware of subsurfaces and use the
transient_for which represents the actual parent (whereas the parent
might be pointing to the root window).

Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=774148
This commit is contained in:
Olivier Fourdan
2017-02-01 15:09:52 +01:00
parent 03429df872
commit e5b6375914
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
gdk/gdkwindow.c
View File

@ -637,6 +637,12 @@ gdk_window_is_offscreen (GdkWindow *window)
return window->window_type == GDK_WINDOW_OFFSCREEN;
}
static gboolean
gdk_window_is_subsurface (GdkWindow *window)
{
return window->window_type == GDK_WINDOW_SUBSURFACE;
}
static GdkWindow *
gdk_window_get_impl_window (GdkWindow *window)
{
@ -2429,6 +2435,8 @@ gdk_window_get_effective_parent (GdkWindow *window)
if (gdk_window_is_offscreen (window))
return gdk_offscreen_window_get_embedder (window);
else if (gdk_window_is_subsurface (window))
return window->transient_for;
else
return window->parent;
}