From 5f9555763147a2b963011ee70b09499283ad10a3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Kristian=20H=C3=B8gsberg?= <krh@bitplanet.net>
Date: Tue, 7 May 2013 21:06:38 -0400
Subject: [PATCH] data-device: Setup and release drag surface and beginning and
 end of drag

Previously we just got the drag_icon signal and had to figure out what
changed.  Now we can directly setup or release the drag icon when the
drag starts and stops.
---
 src/compositor.h  |  2 --
 src/data-device.c | 41 ++++-------------------------------------
 2 files changed, 4 insertions(+), 39 deletions(-)

diff --git a/src/compositor.h b/src/compositor.h
index 335f2ee7..5a9349d9 100644
--- a/src/compositor.h
+++ b/src/compositor.h
@@ -472,10 +472,8 @@ struct weston_seat {
 	struct wl_resource *drag_focus_resource;
 	struct wl_listener drag_focus_listener;
 	struct weston_pointer_grab drag_grab;
-	struct wl_surface *next_drag_surface;
 	struct weston_surface *drag_surface;
 	struct wl_listener drag_surface_destroy_listener;
-	struct wl_listener drag_icon_listener;
 
 	uint32_t num_tp;
 
diff --git a/src/data-device.c b/src/data-device.c
index c4f8e3d8..16ee1213 100644
--- a/src/data-device.c
+++ b/src/data-device.c
@@ -231,30 +231,9 @@ device_map_drag_surface(struct weston_seat *seat)
 void
 weston_seat_update_drag_surface(struct weston_seat *seat, int dx, int dy)
 {
-	int surface_changed = 0;
-
-	if (!seat->drag_surface && !seat->next_drag_surface)
+	if (!seat->drag_surface)
 		return;
 
-	if (seat->drag_surface && seat->next_drag_surface &&
-	    (&seat->drag_surface->surface.resource !=
-	     &seat->next_drag_surface->resource))
-		/* between calls to this funcion we got a new drag_surface */
-		surface_changed = 1;
-
-	if (!seat->next_drag_surface || surface_changed) {
-		device_release_drag_surface(seat);
-		if (!surface_changed)
-			return;
-	}
-
-	if (!seat->drag_surface || surface_changed) {
-		struct weston_surface *surface =
-			(struct weston_surface *) seat->next_drag_surface;
-		if (!device_setup_new_drag_surface(seat, surface))
-			return;
-	}
-
 	/* the client may not have attached a buffer to the drag surface
 	 * when we setup it up, so check if map is needed on every update */
 	device_map_drag_surface(seat);
@@ -348,9 +327,8 @@ static void
 data_device_end_drag_grab(struct weston_seat *seat)
 {
 	if (seat->drag_surface) {
-		seat->next_drag_surface = NULL;
+		device_release_drag_surface(seat);
 		weston_seat_update_drag_surface(seat, 0, 0);
-		wl_list_remove(&seat->drag_icon_listener.link);
 	}
 
 	drag_grab_focus(&seat->drag_grab, NULL,
@@ -398,15 +376,6 @@ destroy_data_device_source(struct wl_listener *listener, void *data)
 	data_device_end_drag_grab(seat);
 }
 
-static void
-destroy_data_device_icon(struct wl_listener *listener, void *data)
-{
-	struct weston_seat *seat = container_of(listener, struct weston_seat,
-						drag_icon_listener);
-
-	seat->next_drag_surface = NULL;
-}
-
 static void
 handle_drag_surface_destroy(struct wl_listener *listener, void *data)
 {
@@ -447,10 +416,8 @@ data_device_start_drag(struct wl_client *client, struct wl_resource *resource,
 	}
 
 	if (icon_resource) {
-		seat->next_drag_surface = icon_resource->data;
-		seat->drag_icon_listener.notify = destroy_data_device_icon;
-		wl_signal_add(&icon_resource->destroy_signal,
-			      &seat->drag_icon_listener);
+		if (!device_setup_new_drag_surface(seat, icon_resource->data))
+			return;
 		weston_seat_update_drag_surface(seat, 0, 0);
 	}