[gtk+/wip/carlosg/event-delivery-cleanups: 29/30] gdk: Coalesce 2 GdkWindow fields in GdkPointerWindowInfo struct
- From: Carlos Garnacho <carlosg src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gtk+/wip/carlosg/event-delivery-cleanups: 29/30] gdk: Coalesce 2 GdkWindow fields in GdkPointerWindowInfo struct
- Date: Fri, 15 Sep 2017 19:12:19 +0000 (UTC)
commit 8adabe528250f734e8691ae31bfddf77dea79b36
Author: Carlos Garnacho <carlosg gnome org>
Date: Fri Sep 15 19:39:18 2017 +0200
gdk: Coalesce 2 GdkWindow fields in GdkPointerWindowInfo struct
One used to point to the toplevel and the other to the client-side window
that the pointer pointed to. The latter was made to be like the former in
most places, so put those together, and fix the remaining cases where the
variable might not end up with a toplevel/native window.
gdk/gdkdisplay.c | 29 ++++++-----------------------
gdk/gdkdisplayprivate.h | 2 --
gdk/gdkwindow.c | 6 +++---
3 files changed, 9 insertions(+), 28 deletions(-)
---
diff --git a/gdk/gdkdisplay.c b/gdk/gdkdisplay.c
index ad83511..4c9454f 100644
--- a/gdk/gdkdisplay.c
+++ b/gdk/gdkdisplay.c
@@ -346,8 +346,7 @@ gdk_display_class_init (GdkDisplayClass *class)
static void
free_pointer_info (GdkPointerWindowInfo *info)
{
- if (info->toplevel_under_pointer)
- g_object_unref (info->toplevel_under_pointer);
+ g_clear_object (&info->window_under_pointer);
g_slice_free (GdkPointerWindowInfo, info);
}
@@ -736,7 +735,7 @@ switch_to_pointer_grab (GdkDisplay *display,
guint32 time,
gulong serial)
{
- GdkWindow *pointer_window, *new_toplevel;
+ GdkWindow *new_toplevel;
GdkPointerWindowInfo *info;
GList *old_grabs;
GdkModifierType state;
@@ -768,13 +767,6 @@ switch_to_pointer_grab (GdkDisplay *display,
if (grab == NULL /* ungrab */ ||
(!last_grab->owner_events && grab->owner_events) /* switched to owner_events */ )
{
- /* We force check what window we're in, and update the toplevel_under_pointer info,
- * as that won't get told of this change with toplevel enter events.
- */
- if (info->toplevel_under_pointer)
- g_object_unref (info->toplevel_under_pointer);
- info->toplevel_under_pointer = NULL;
-
/* Ungrabbed slave devices don't have a position by
* itself, rather depend on its master pointer, so
* it doesn't make sense to track any position for
@@ -786,7 +778,7 @@ switch_to_pointer_grab (GdkDisplay *display,
if (new_toplevel)
{
/* w is now toplevel and x,y in toplevel coords */
- info->toplevel_under_pointer = g_object_ref (new_toplevel);
+ _gdk_display_set_window_under_pointer (display, device, new_toplevel);
info->toplevel_x = x;
info->toplevel_y = y;
info->state = state;
@@ -803,20 +795,11 @@ switch_to_pointer_grab (GdkDisplay *display,
(gdk_device_get_source (source_device) == GDK_SOURCE_TOUCHSCREEN))
info->need_touch_press_enter = TRUE;
- pointer_window = NULL;
-
- if (new_toplevel &&
- !info->need_touch_press_enter)
- {
- /* Find (possibly virtual) child window */
- pointer_window =
- _gdk_window_find_descendant_at (new_toplevel,
- x, y,
- NULL, NULL);
- }
+ if (info->need_touch_press_enter)
+ new_toplevel = NULL;
/* We're now ungrabbed, update the window_under_pointer */
- _gdk_display_set_window_under_pointer (display, device, pointer_window);
+ _gdk_display_set_window_under_pointer (display, device, new_toplevel);
}
}
diff --git a/gdk/gdkdisplayprivate.h b/gdk/gdkdisplayprivate.h
index ee5d694..de280c4 100644
--- a/gdk/gdkdisplayprivate.h
+++ b/gdk/gdkdisplayprivate.h
@@ -63,8 +63,6 @@ typedef struct
*/
typedef struct
{
- GdkWindow *toplevel_under_pointer; /* toplevel window containing the pointer, */
- /* tracked via native events */
GdkWindow *window_under_pointer; /* window that last got a normal enter event */
gdouble toplevel_x, toplevel_y;
guint32 state;
diff --git a/gdk/gdkwindow.c b/gdk/gdkwindow.c
index 785b3d0..b99bc1b 100644
--- a/gdk/gdkwindow.c
+++ b/gdk/gdkwindow.c
@@ -1313,10 +1313,10 @@ update_pointer_info_foreach (GdkDisplay *display,
{
GdkWindow *window = user_data;
- if (pointer_info->toplevel_under_pointer == window)
+ if (pointer_info->window_under_pointer == window)
{
- g_object_unref (pointer_info->toplevel_under_pointer);
- pointer_info->toplevel_under_pointer = NULL;
+ g_object_unref (pointer_info->window_under_pointer);
+ pointer_info->window_under_pointer = NULL;
}
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]