summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthias Clasen <mclasen@redhat.com>2019-03-23 00:14:00 -0400
committerMatthias Clasen <mclasen@redhat.com>2019-03-26 19:18:39 -0400
commit19919365de195cc361d34cb2ca3f7bc9bb69ff95 (patch)
tree849bf8dd1d1ef1325b53d12352ab3f8da32d4b13
parent39a7f4765e05722f90beda4bc1bbe2dfde4afc05 (diff)
downloadgtk+-19919365de195cc361d34cb2ca3f7bc9bb69ff95.tar.gz
x11: Stop using child surface api
We no longer have child surfaces. Drop code that only makes sense in that case.
-rw-r--r--gdk/x11/gdkdevicemanager-core-x11.c29
-rw-r--r--gdk/x11/gdkdevicemanager-xi2.c24
-rw-r--r--gdk/x11/gdksurface-x11.c2
-rw-r--r--gdk/x11/gdkxid.c5
4 files changed, 8 insertions, 52 deletions
diff --git a/gdk/x11/gdkdevicemanager-core-x11.c b/gdk/x11/gdkdevicemanager-core-x11.c
index 305bec3d6f..8881fc8d0c 100644
--- a/gdk/x11/gdkdevicemanager-core-x11.c
+++ b/gdk/x11/gdkdevicemanager-core-x11.c
@@ -258,17 +258,14 @@ static void
set_user_time (GdkSurface *surface,
GdkEvent *event)
{
- g_return_if_fail (event != NULL);
-
- surface = gdk_surface_get_toplevel (event->any.surface);
g_return_if_fail (GDK_IS_SURFACE (surface));
+ g_return_if_fail (event != NULL);
/* If an event doesn't have a valid timestamp, we shouldn't use it
* to update the latest user interaction time.
*/
if (gdk_event_get_time (event) != GDK_CURRENT_TIME)
- gdk_x11_surface_set_user_time (gdk_surface_get_toplevel (surface),
- gdk_event_get_time (event));
+ gdk_x11_surface_set_user_time (surface, gdk_event_get_time (event));
}
static GdkCrossingMode
@@ -309,24 +306,6 @@ translate_notify_type (int detail)
}
}
-static gboolean
-is_parent_of (GdkSurface *parent,
- GdkSurface *child)
-{
- GdkSurface *w;
-
- w = child;
- while (w != NULL)
- {
- if (w == parent)
- return TRUE;
-
- w = gdk_surface_get_parent (w);
- }
-
- return FALSE;
-}
-
static GdkSurface *
get_event_surface (GdkEventTranslator *translator,
const XEvent *xevent)
@@ -347,9 +326,7 @@ get_event_surface (GdkEventTranslator *translator,
info = _gdk_display_has_device_grab (display,
GDK_X11_DEVICE_MANAGER_CORE (translator)->core_keyboard,
serial);
- if (info &&
- (!is_parent_of (info->surface, surface) ||
- !info->owner_events))
+ if (info && !info->owner_events)
{
/* Report key event against grab surface */
surface = info->surface;
diff --git a/gdk/x11/gdkdevicemanager-xi2.c b/gdk/x11/gdkdevicemanager-xi2.c
index 01d51d3d3a..0f0f0aadce 100644
--- a/gdk/x11/gdkdevicemanager-xi2.c
+++ b/gdk/x11/gdkdevicemanager-xi2.c
@@ -1163,7 +1163,7 @@ set_user_time (GdkEvent *event)
GdkSurface *surface;
guint32 time;
- surface = gdk_surface_get_toplevel (event->any.surface);
+ surface = event->any.surface;
g_return_if_fail (GDK_IS_SURFACE (surface));
time = gdk_event_get_time (event);
@@ -1230,24 +1230,6 @@ translate_axes (GdkDevice *device,
}
static gboolean
-is_parent_of (GdkSurface *parent,
- GdkSurface *child)
-{
- GdkSurface *w;
-
- w = child;
- while (w != NULL)
- {
- if (w == parent)
- return TRUE;
-
- w = gdk_surface_get_parent (w);
- }
-
- return FALSE;
-}
-
-static gboolean
get_event_surface (GdkEventTranslator *translator,
XIEvent *ev,
GdkSurface **surface_p)
@@ -1288,9 +1270,7 @@ get_event_surface (GdkEventTranslator *translator,
serial = _gdk_display_get_next_serial (display);
info = _gdk_display_has_device_grab (display, device, serial);
- if (info &&
- (!is_parent_of (info->surface, surface) ||
- !info->owner_events))
+ if (info && !info->owner_events)
{
/* Report key event against grab surface */
surface = info->surface;
diff --git a/gdk/x11/gdksurface-x11.c b/gdk/x11/gdksurface-x11.c
index a8a8140b2e..22024f016d 100644
--- a/gdk/x11/gdksurface-x11.c
+++ b/gdk/x11/gdksurface-x11.c
@@ -227,7 +227,7 @@ set_sync_counter(Display *display,
void
gdk_x11_surface_pre_damage (GdkSurface *surface)
{
- GdkSurface *toplevel_surface = gdk_surface_get_toplevel (surface);
+ GdkSurface *toplevel_surface = surface;
GdkSurfaceImplX11 *impl;
if (!toplevel_surface || !SURFACE_IS_TOPLEVEL (toplevel_surface))
diff --git a/gdk/x11/gdkxid.c b/gdk/x11/gdkxid.c
index 93ca3c12a3..b08ffce7bf 100644
--- a/gdk/x11/gdkxid.c
+++ b/gdk/x11/gdkxid.c
@@ -62,8 +62,7 @@ _gdk_x11_display_add_window (GdkDisplay *display,
g_hash_table_insert (display_x11->xid_ht, xid, data);
- if (gdk_surface_get_parent (GDK_SURFACE (data)) == NULL)
- display_x11->toplevels = g_list_prepend (display_x11->toplevels, data);
+ display_x11->toplevels = g_list_prepend (display_x11->toplevels, data);
}
void
@@ -81,7 +80,7 @@ _gdk_x11_display_remove_window (GdkDisplay *display,
return;
surface = g_hash_table_lookup (display_x11->xid_ht, &xid);
- if (surface && gdk_surface_get_parent (surface) == NULL)
+ if (surface)
display_x11->toplevels = g_list_remove (display_x11->toplevels, surface);
g_hash_table_remove (display_x11->xid_ht, &xid);