data-device: refactor wlr_drag

10 files changed, 275 insertions, 150 deletions

diff --git a/include/rootston/seat.h b/include/rootston/seat.h
index a91d8481..b91f9578 100644
--- a/include/rootston/seat.h
+++ b/include/rootston/seat.h
@@ -40,7 +40,8 @@ struct roots_seat {
 
 	struct wl_listener request_set_selection;
 	struct wl_listener request_set_primary_selection;
-	struct wl_listener new_drag_icon;
+	struct wl_listener request_start_drag;
+	struct wl_listener start_drag;
 	struct wl_listener destroy;
 };
 
diff --git a/include/types/wlr_data_device.h b/include/types/wlr_data_device.h
index 28073880..5865c999 100644
--- a/include/types/wlr_data_device.h
+++ b/include/types/wlr_data_device.h
@@ -30,9 +30,6 @@ void data_source_notify_finish(struct wlr_data_source *source);
 
 struct wlr_seat_client *seat_client_from_data_device_resource(
 	struct wl_resource *resource);
-bool seat_client_start_drag(struct wlr_seat_client *client,
-	struct wlr_data_source *source, struct wlr_surface *icon_surface,
-	struct wlr_surface *origin, uint32_t serial);
 /**
  * Creates a new wl_data_offer if there is a wl_data_source currently set as
  * the seat selection and sends it to the seat client, followed by the
diff --git a/include/wlr/types/wlr_data_device.h b/include/wlr/types/wlr_data_device.h
index 59aa718f..9da7cc0d 100644
--- a/include/wlr/types/wlr_data_device.h
+++ b/include/wlr/types/wlr_data_device.h
@@ -89,15 +89,13 @@ struct wlr_data_source {
 	} events;
 };
 
+struct wlr_drag;
+
 struct wlr_drag_icon {
+	struct wlr_drag *drag;
 	struct wlr_surface *surface;
-	struct wlr_seat_client *client;
-	struct wl_list link; // wlr_seat::drag_icons
 	bool mapped;
 
-	bool is_pointer;
-	int32_t touch_id;
-
 	struct {
 		struct wl_signal map;
 		struct wl_signal unmap;
@@ -105,40 +103,46 @@ struct wlr_drag_icon {
 	} events;
 
 	struct wl_listener surface_destroy;
-	struct wl_listener seat_client_destroy;
 
 	void *data;
 };
 
+enum wlr_drag_grab_type {
+	WLR_DRAG_GRAB_KEYBOARD,
+	WLR_DRAG_GRAB_KEYBOARD_POINTER,
+	WLR_DRAG_GRAB_KEYBOARD_TOUCH,
+};
+
 struct wlr_drag {
-	struct wlr_seat_pointer_grab pointer_grab;
+	enum wlr_drag_grab_type grab_type;
 	struct wlr_seat_keyboard_grab keyboard_grab;
+	struct wlr_seat_pointer_grab pointer_grab;
 	struct wlr_seat_touch_grab touch_grab;
 
 	struct wlr_seat *seat;
 	struct wlr_seat_client *seat_client;
 	struct wlr_seat_client *focus_client;
 
-	bool is_pointer_grab;
+	struct wlr_drag_icon *icon; // can be NULL
+	struct wlr_surface *focus; // can be NULL
+	struct wlr_data_source *source; // can be NULL
 
-	struct wlr_drag_icon *icon;
-	struct wlr_surface *focus;
-	struct wlr_data_source *source;
+	bool started, cancelling;
+	int32_t grab_touch_id, touch_id; // if WLR_DRAG_GRAB_TOUCH
 
-	bool cancelling;
-	int32_t grab_touch_id;
+	struct {
+		struct wl_signal focus;
+		struct wl_signal motion; // wlr_drag_motion_event
+		struct wl_signal drop; // wlr_drag_drop_event
+		struct wl_signal destroy;
+	} events;
 
 	struct wl_listener point_destroy;
 	struct wl_listener source_destroy;
 	struct wl_listener seat_client_destroy;
 	struct wl_listener icon_destroy;
 
-	struct {
-		struct wl_signal focus;
-		struct wl_signal motion;
-		struct wl_signal drop;
-		struct wl_signal destroy;
-	} events;
+	void *data;
 };
 
 struct wlr_drag_motion_event {
@@ -179,6 +183,40 @@ void wlr_seat_set_selection(struct wlr_seat *seat,
 	struct wlr_data_source *source, uint32_t serial);
 
 /**
+ * Creates a new drag. To request to start the drag, call
+ * `wlr_seat_request_start_drag`.
+ */
+struct wlr_drag *wlr_drag_create(struct wlr_seat_client *seat_client,
+	struct wlr_data_source *source, struct wlr_surface *icon_surface);
+
+/**
+ * Requests a drag to be started on the seat.
+ */
+void wlr_seat_request_start_drag(struct wlr_seat *seat, struct wlr_drag *drag,
+	struct wlr_surface *origin, uint32_t serial);
+
+/**
+ * Starts a drag on the seat. This starts an implicit keyboard grab, but doesn't
+ * start a pointer or a touch grab.
+ */
+void wlr_seat_start_drag(struct wlr_seat *seat, struct wlr_drag *drag,
+	uint32_t serial);
+
+/**
+ * Starts a pointer drag on the seat. This starts implicit keyboard and pointer
+ * grabs.
+ */
+void wlr_seat_start_pointer_drag(struct wlr_seat *seat, struct wlr_drag *drag,
+	uint32_t serial);
+
+/**
+ * Starts a touch drag on the seat. This starts implicit keyboard and touch
+ * grabs.
+ */
+void wlr_seat_start_touch_drag(struct wlr_seat *seat, struct wlr_drag *drag,
+	uint32_t serial, struct wlr_touch_point *point);
+
+/**
  * Initializes the data source with the provided implementation.
  */
 void wlr_data_source_init(struct wlr_data_source *source,
diff --git a/include/wlr/types/wlr_seat.h b/include/wlr/types/wlr_seat.h
index c81c2cec..c020dbdb 100644
--- a/include/wlr/types/wlr_seat.h
+++ b/include/wlr/types/wlr_seat.h
@@ -191,7 +191,6 @@ struct wlr_seat {
 	struct wl_global *global;
 	struct wl_display *display;
 	struct wl_list clients;
-	struct wl_list drag_icons; // wlr_drag_icon::link
 
 	char *name;
 	uint32_t capabilities;
@@ -239,8 +238,9 @@ struct wlr_seat {
 		struct wl_signal request_set_primary_selection;
 		struct wl_signal set_primary_selection;
 
+		// wlr_seat_request_start_drag_event
+		struct wl_signal request_start_drag;
 		struct wl_signal start_drag; // wlr_drag
-		struct wl_signal new_drag_icon; // wlr_drag_icon
 
 		struct wl_signal destroy;
 	} events;
@@ -265,6 +265,12 @@ struct wlr_seat_request_set_primary_selection_event {
 	uint32_t serial;
 };
 
+struct wlr_seat_request_start_drag_event {
+	struct wlr_drag *drag;
+	struct wlr_surface *origin;
+	uint32_t serial;
+};
+
 struct wlr_seat_pointer_focus_change_event {
 	struct wlr_seat *seat;
 	struct wlr_surface *old_surface, *new_surface;
@@ -597,9 +603,30 @@ bool wlr_seat_touch_has_grab(struct wlr_seat *seat);
  */
 bool wlr_seat_validate_grab_serial(struct wlr_seat *seat, uint32_t serial);
 
+/**
+ * Check whether this serial is valid to start a pointer grab action.
+ */
+bool wlr_seat_validate_pointer_grab_serial(struct wlr_seat *seat,
+	struct wlr_surface *origin, uint32_t serial);
+
+/**
+ * Check whether this serial is valid to start a touch grab action. If it's the
+ * case and point_ptr is non-NULL, *point_ptr is set to the touch point matching
+ * the serial.
+ */
+bool wlr_seat_validate_touch_grab_serial(struct wlr_seat *seat,
+	struct wlr_surface *origin, uint32_t serial,
+	struct wlr_touch_point **point_ptr);
+
+/**
+ * Get a seat client from a seat resource. Returns NULL if inert.
+ */
 struct wlr_seat_client *wlr_seat_client_from_resource(
 	struct wl_resource *resource);
 
+/**
+ * Get a seat client from a pointer resource. Returns NULL if inert.
+ */
 struct wlr_seat_client *wlr_seat_client_from_pointer_resource(
 	struct wl_resource *resource);
 
diff --git a/rootston/seat.c b/rootston/seat.c
index cd95472d..428787ef 100644
--- a/rootston/seat.c
+++ b/rootston/seat.c
@@ -603,10 +603,37 @@ static void roots_drag_icon_handle_destroy(struct wl_listener *listener,
 	free(icon);
 }
 
-static void roots_seat_handle_new_drag_icon(struct wl_listener *listener,
+static void roots_seat_handle_request_start_drag(struct wl_listener *listener,
 		void *data) {
-	struct roots_seat *seat = wl_container_of(listener, seat, new_drag_icon);
-	struct wlr_drag_icon *wlr_drag_icon = data;
+	struct roots_seat *seat =
+		wl_container_of(listener, seat, request_start_drag);
+	struct wlr_seat_request_start_drag_event *event = data;
+
+	if (wlr_seat_validate_pointer_grab_serial(seat->seat,
+			event->origin, event->serial)) {
+		wlr_seat_start_pointer_drag(seat->seat, event->drag, event->serial);
+		return;
+	}
+
+	struct wlr_touch_point *point;
+	if (wlr_seat_validate_touch_grab_serial(seat->seat,
+			event->origin, event->serial, &point)) {
+		wlr_seat_start_touch_drag(seat->seat, event->drag, event->serial, point);
+		return;
+	}
+
+	wlr_log(WLR_DEBUG, "Ignoring start_drag request: "
+		"could not validate pointer or touch serial %" PRIu32, event->serial);
+}
+
+static void roots_seat_handle_start_drag(struct wl_listener *listener,
+		void *data) {
+	struct roots_seat *seat = wl_container_of(listener, seat, start_drag);
+	struct wlr_drag *wlr_drag = data;
+	struct wlr_drag_icon *wlr_drag_icon = wlr_drag->icon;
+	if (wlr_drag_icon == NULL) {
+		return;
+	}
 
 	struct roots_drag_icon *icon = calloc(1, sizeof(struct roots_drag_icon));
 	if (icon == NULL) {
@@ -649,20 +676,27 @@ static void roots_seat_handle_request_set_primary_selection(
 void roots_drag_icon_update_position(struct roots_drag_icon *icon) {
 	roots_drag_icon_damage_whole(icon);
 
-	struct wlr_drag_icon *wlr_icon = icon->wlr_drag_icon;
 	struct roots_seat *seat = icon->seat;
-	struct wlr_cursor *cursor = seat->cursor->cursor;
-	if (wlr_icon->is_pointer) {
+	struct wlr_drag *wlr_drag = icon->wlr_drag_icon->drag;
+	assert(wlr_drag != NULL);
+
+	switch (seat->seat->drag->grab_type) {
+	case WLR_DRAG_GRAB_KEYBOARD:
+		assert(false);
+	case WLR_DRAG_GRAB_KEYBOARD_POINTER:;
+		struct wlr_cursor *cursor = seat->cursor->cursor;
 		icon->x = cursor->x;
 		icon->y = cursor->y;
-	} else {
+		break;
+	case WLR_DRAG_GRAB_KEYBOARD_TOUCH:;
 		struct wlr_touch_point *point =
-			wlr_seat_touch_get_point(seat->seat, wlr_icon->touch_id);
+			wlr_seat_touch_get_point(seat->seat, wlr_drag->touch_id);
 		if (point == NULL) {
 			return;
 		}
 		icon->x = seat->touch_x;
 		icon->y = seat->touch_y;
+		break;
 	}
 
 	roots_drag_icon_damage_whole(icon);
@@ -738,8 +772,11 @@ struct roots_seat *roots_seat_create(struct roots_input *input, char *name) {
 		roots_seat_handle_request_set_primary_selection;
 	wl_signal_add(&seat->seat->events.request_set_primary_selection,
 		&seat->request_set_primary_selection);
-	seat->new_drag_icon.notify = roots_seat_handle_new_drag_icon;
-	wl_signal_add(&seat->seat->events.new_drag_icon, &seat->new_drag_icon);
+	seat->request_start_drag.notify = roots_seat_handle_request_start_drag;
+	wl_signal_add(&seat->seat->events.request_start_drag,
+		&seat->request_start_drag);
+	seat->start_drag.notify = roots_seat_handle_start_drag;
+	wl_signal_add(&seat->seat->events.start_drag, &seat->start_drag);
 	seat->destroy.notify = roots_seat_handle_destroy;
 	wl_signal_add(&seat->seat->events.destroy, &seat->destroy);
 
diff --git a/types/data_device/wlr_data_device.c b/types/data_device/wlr_data_device.c
index 36a9d1b8..d494cf75 100644
--- a/types/data_device/wlr_data_device.c
+++ b/types/data_device/wlr_data_device.c
@@ -73,8 +73,8 @@ static void data_device_start_drag(struct wl_client *client,
 
 	struct wlr_data_source *wlr_source =
 		source != NULL ? &source->source : NULL;
-	if (!seat_client_start_drag(seat_client, wlr_source, icon,
-			origin, serial)) {
+	struct wlr_drag *drag = wlr_drag_create(seat_client, wlr_source, icon);
+	if (drag == NULL) {
 		wl_resource_post_no_memory(device_resource);
 		return;
 	}
@@ -82,6 +82,8 @@ static void data_device_start_drag(struct wl_client *client,
 	if (source != NULL) {
 		source->finalized = true;
 	}
+
+	wlr_seat_request_start_drag(seat_client->seat, drag, origin, serial);
 }
 
 static void data_device_release(struct wl_client *client,
diff --git a/types/data_device/wlr_drag.c b/types/data_device/wlr_drag.c
index fdfabdcf..3514a26c 100644
--- a/types/data_device/wlr_drag.c
+++ b/types/data_device/wlr_drag.c
@@ -101,33 +101,39 @@ static void drag_icon_set_mapped(struct wlr_drag_icon *icon, bool mapped) {
 	}
 }
 
-static void drag_end(struct wlr_drag *drag) {
-	if (!drag->cancelling) {
-		drag->cancelling = true;
-		if (drag->is_pointer_grab) {
-			wlr_seat_pointer_end_grab(drag->seat);
-		} else {
-			wlr_seat_touch_end_grab(drag->seat);
-		}
-		wlr_seat_keyboard_end_grab(drag->seat);
+static void drag_icon_destroy(struct wlr_drag_icon *icon);
 
-		if (drag->source) {
-			wl_list_remove(&drag->source_destroy.link);
-		}
+static void drag_end(struct wlr_drag *drag) {
+	if (drag->cancelling) {
+		return;
+	}
+	drag->cancelling = true;
+
+	wlr_signal_emit_safe(&drag->events.destroy, drag);
+
+	wlr_seat_keyboard_end_grab(drag->seat);
+	switch (drag->grab_type) {
+	case WLR_DRAG_GRAB_KEYBOARD:
+		break;
+	case WLR_DRAG_GRAB_KEYBOARD_POINTER:
+		wlr_seat_pointer_end_grab(drag->seat);
+		break;
+	case WLR_DRAG_GRAB_KEYBOARD_TOUCH:
+		wlr_seat_touch_end_grab(drag->seat);
+		break;
+	}
 
-		drag_set_focus(drag, NULL, 0, 0);
+	if (drag->source) {
+		wl_list_remove(&drag->source_destroy.link);
+	}
 
-		if (drag->icon) {
-			wl_list_remove(&drag->icon_destroy.link);
-			drag_icon_set_mapped(drag->icon, false);
-		}
+	drag_set_focus(drag, NULL, 0, 0);
+	drag_icon_destroy(drag->icon);
 
-		assert(drag->seat->drag == drag);
-		drag->seat->drag = NULL;
+	assert(drag->seat->drag == drag);
+	drag->seat->drag = NULL;
 
-		wlr_signal_emit_safe(&drag->events.destroy, drag);
-		free(drag);
-	}
+	free(drag);
 }
 
 static void drag_handle_pointer_enter(struct wlr_seat_pointer_grab *grab,
@@ -318,8 +324,6 @@ static void drag_icon_destroy(struct wlr_drag_icon *icon) {
 	wlr_signal_emit_safe(&icon->events.destroy, icon);
 	icon->surface->role_data = NULL;
 	wl_list_remove(&icon->surface_destroy.link);
-	wl_list_remove(&icon->seat_client_destroy.link);
-	wl_list_remove(&icon->link);
 	free(icon);
 }
 
@@ -345,26 +349,15 @@ const struct wlr_surface_role drag_icon_surface_role = {
 	.commit = drag_icon_surface_role_commit,
 };
 
-static void drag_icon_handle_seat_client_destroy(struct wl_listener *listener,
-		void *data) {
-	struct wlr_drag_icon *icon =
-		wl_container_of(listener, icon, seat_client_destroy);
-
-	drag_icon_destroy(icon);
-}
-
-static struct wlr_drag_icon *drag_icon_create(
-		struct wlr_surface *icon_surface, struct wlr_seat_client *client,
-		bool is_pointer, int32_t touch_id) {
+static struct wlr_drag_icon *drag_icon_create(struct wlr_drag *drag,
+		struct wlr_surface *surface) {
 	struct wlr_drag_icon *icon = calloc(1, sizeof(struct wlr_drag_icon));
 	if (!icon) {
 		return NULL;
 	}
 
-	icon->surface = icon_surface;
-	icon->client = client;
-	icon->is_pointer = is_pointer;
-	icon->touch_id = touch_id;
+	icon->drag = drag;
+	icon->surface = surface;
 
 	wl_signal_init(&icon->events.map);
 	wl_signal_init(&icon->events.unmap);
@@ -373,15 +366,9 @@ static struct wlr_drag_icon *drag_icon_create(
 	wl_signal_add(&icon->surface->events.destroy, &icon->surface_destroy);
 	icon->surface_destroy.notify = drag_icon_handle_surface_destroy;
 
-	wl_signal_add(&client->events.destroy, &icon->seat_client_destroy);
-	icon->seat_client_destroy.notify = drag_icon_handle_seat_client_destroy;
-
 	icon->surface->role_data = icon;
 
-	wl_list_insert(&client->seat->drag_icons, &icon->link);
-	wlr_signal_emit_safe(&client->seat->events.new_drag_icon, icon);
-
-	if (wlr_surface_has_buffer(icon_surface)) {
+	if (wlr_surface_has_buffer(surface)) {
 		drag_icon_set_mapped(icon, true);
 	}
 
@@ -389,20 +376,11 @@ static struct wlr_drag_icon *drag_icon_create(
 }
 
 
-static void seat_handle_drag_source_destroy(struct wl_listener *listener,
-		void *data) {
-	struct wlr_seat *seat =
-		wl_container_of(listener, seat, drag_source_destroy);
-	wl_list_remove(&seat->drag_source_destroy.link);
-	seat->drag_source = NULL;
-}
-
-bool seat_client_start_drag(struct wlr_seat_client *client,
-		struct wlr_data_source *source, struct wlr_surface *icon_surface,
-		struct wlr_surface *origin, uint32_t serial) {
+struct wlr_drag *wlr_drag_create(struct wlr_seat_client *seat_client,
+		struct wlr_data_source *source, struct wlr_surface *icon_surface) {
 	struct wlr_drag *drag = calloc(1, sizeof(struct wlr_drag));
 	if (drag == NULL) {
-		return false;
+		return NULL;
 	}
 
 	wl_signal_init(&drag->events.focus);
@@ -410,51 +388,18 @@ bool seat_client_start_drag(struct wlr_seat_client *client,
 	wl_signal_init(&drag->events.drop);
 	wl_signal_init(&drag->events.destroy);
 
-	struct wlr_seat *seat = client->seat;
-	drag->seat = seat;
-
-	drag->is_pointer_grab = !wl_list_empty(&client->pointers) &&
-		seat->pointer_state.button_count == 1 &&
-		seat->pointer_state.grab_serial == serial &&
-		seat->pointer_state.focused_surface &&
-		seat->pointer_state.focused_surface == origin;
-
-	bool is_touch_grab = !wl_list_empty(&client->touches) &&
-		wlr_seat_touch_num_points(seat) == 1 &&
-		seat->touch_state.grab_serial == serial;
-
-	// set in the iteration
-	struct wlr_touch_point *point = NULL;
-	if (is_touch_grab) {
-		wl_list_for_each(point, &seat->touch_state.touch_points, link) {
-			is_touch_grab = point->surface && point->surface == origin;
-			break;
-		}
-	}
-
-	if (!drag->is_pointer_grab && !is_touch_grab) {
-		free(drag);
-		return true;
-	}
-
-	if (seat->drag != NULL) {
-		wlr_log(WLR_DEBUG, "Refusing to start drag, "
-			"another drag is already in progress");
-		free(drag);
-		return true;
-	}
+	drag->seat = seat_client->seat;
+	drag->seat_client = seat_client;
 
 	if (icon_surface) {
-		int32_t touch_id = (point ? point->touch_id : 0);
-		struct wlr_drag_icon *icon =
-			drag_icon_create(icon_surface, client, drag->is_pointer_grab,
-				touch_id);
-		if (!icon) {
+		struct wlr_drag_icon *icon = drag_icon_create(drag, icon_surface);
+		if (icon == NULL) {
 			free(drag);
-			return false;
+			return NULL;
 		}
 
 		drag->icon = icon;
+
 		drag->icon_destroy.notify = drag_handle_icon_destroy;
 		wl_signal_add(&icon->events.destroy, &drag->icon_destroy);
 	}
@@ -465,41 +410,85 @@ bool seat_client_start_drag(struct wlr_seat_client *client,
 		wl_signal_add(&source->events.destroy, &drag->source_destroy);
 	}
 
-	drag->seat_client = client;
 	drag->pointer_grab.data = drag;
 	drag->pointer_grab.interface = &data_device_pointer_drag_interface;
 
 	drag->touch_grab.data = drag;
 	drag->touch_grab.interface = &data_device_touch_drag_interface;
-	drag->grab_touch_id = seat->touch_state.grab_id;
 
 	drag->keyboard_grab.data = drag;
 	drag->keyboard_grab.interface = &data_device_keyboard_drag_interface;
 
-	wlr_seat_keyboard_start_grab(seat, &drag->keyboard_grab);
+	return drag;
+}
+
+void wlr_seat_request_start_drag(struct wlr_seat *seat, struct wlr_drag *drag,
+		struct wlr_surface *origin, uint32_t serial) {
+	assert(drag->seat == seat);
 
-	if (drag->is_pointer_grab) {
-		wlr_seat_pointer_clear_focus(seat);
-		wlr_seat_pointer_start_grab(seat, &drag->pointer_grab);
-	} else {
-		assert(point);
-		wlr_seat_touch_start_grab(seat, &drag->touch_grab);
-		drag_set_focus(drag, point->surface, point->sx, point->sy);
+	if (seat->drag != NULL) {
+		wlr_log(WLR_DEBUG, "Rejecting start_drag request, "
+			"another drag-and-drop operation is already in progress");
+		return;
 	}
 
+	struct wlr_seat_request_start_drag_event event = {
+		.drag = drag,
+		.origin = origin,
+		.serial = serial,
+	};
+	wlr_signal_emit_safe(&seat->events.request_start_drag, &event);
+}
+
+static void seat_handle_drag_source_destroy(struct wl_listener *listener,
+		void *data) {
+	struct wlr_seat *seat =
+		wl_container_of(listener, seat, drag_source_destroy);
+	wl_list_remove(&seat->drag_source_destroy.link);
+	seat->drag_source = NULL;
+}
+
+void wlr_seat_start_drag(struct wlr_seat *seat, struct wlr_drag *drag,
+		uint32_t serial) {
+	assert(drag->seat == seat);
+	assert(!drag->started);
+	drag->started = true;
+
+	wlr_seat_keyboard_start_grab(seat, &drag->keyboard_grab);
+
 	seat->drag = drag;
 	seat->drag_serial = serial;
 
 	// We need to destroy the previous source, because listeners only expect one
 	// active drag source at a time.
 	wlr_data_source_destroy(seat->drag_source);
-	seat->drag_source = source;
-	if (source != NULL) {
+	seat->drag_source = drag->source;
+	if (drag->source != NULL) {
 		seat->drag_source_destroy.notify = seat_handle_drag_source_destroy;
-		wl_signal_add(&source->events.destroy, &seat->drag_source_destroy);
+		wl_signal_add(&drag->source->events.destroy, &seat->drag_source_destroy);
 	}
 
 	wlr_signal_emit_safe(&seat->events.start_drag, drag);
+}
+
+void wlr_seat_start_pointer_drag(struct wlr_seat *seat, struct wlr_drag *drag,
+		uint32_t serial) {
+	drag->grab_type = WLR_DRAG_GRAB_KEYBOARD_POINTER;
+
+	wlr_seat_pointer_clear_focus(seat);
+	wlr_seat_pointer_start_grab(seat, &drag->pointer_grab);
+
+	wlr_seat_start_drag(seat, drag, serial);
+}
+
+void wlr_seat_start_touch_drag(struct wlr_seat *seat, struct wlr_drag *drag,
+		uint32_t serial, struct wlr_touch_point *point) {
+	drag->grab_type = WLR_DRAG_GRAB_KEYBOARD_TOUCH;
+	drag->grab_touch_id = seat->touch_state.grab_id;
+	drag->touch_id = point->touch_id;
+
+	wlr_seat_touch_start_grab(seat, &drag->touch_grab);
+	drag_set_focus(drag, point->surface, point->sx, point->sy);
 
-	return true;
+	wlr_seat_start_drag(seat, drag, serial);
 }
diff --git a/types/seat/wlr_seat.c b/types/seat/wlr_seat.c
index 3595be15..b5f1d540 100644
--- a/types/seat/wlr_seat.c
+++ b/types/seat/wlr_seat.c
@@ -266,12 +266,11 @@ struct wlr_seat *wlr_seat_create(struct wl_display *display, const char *name) {
 	seat->display = display;
 	seat->name = strdup(name);
 	wl_list_init(&seat->clients);
-	wl_list_init(&seat->drag_icons);
 	wl_list_init(&seat->selection_offers);
 	wl_list_init(&seat->drag_offers);
 
+	wl_signal_init(&seat->events.request_start_drag);
 	wl_signal_init(&seat->events.start_drag);
-	wl_signal_init(&seat->events.new_drag_icon);
 
 	wl_signal_init(&seat->events.request_set_cursor);
 
diff --git a/types/seat/wlr_seat_pointer.c b/types/seat/wlr_seat_pointer.c
index 31901716..d03385cf 100644
--- a/types/seat/wlr_seat_pointer.c
+++ b/types/seat/wlr_seat_pointer.c
@@ -402,3 +402,17 @@ void seat_client_destroy_pointer(struct wl_resource *resource) {
 	}
 	wl_resource_set_user_data(resource, NULL);
 }
+
+bool wlr_seat_validate_pointer_grab_serial(struct wlr_seat *seat,
+		struct wlr_surface *origin, uint32_t serial) {
+	if (seat->pointer_state.button_count != 1 ||
+			seat->pointer_state.grab_serial != serial) {
+		return false;
+	}
+
+	if (origin != NULL && seat->pointer_state.focused_surface != origin) {
+		return false;
+	}
+
+	return true;
+}
diff --git a/types/seat/wlr_seat_touch.c b/types/seat/wlr_seat_touch.c
index d4fa6f0b..486fa55d 100644
--- a/types/seat/wlr_seat_touch.c
+++ b/types/seat/wlr_seat_touch.c
@@ -359,3 +359,24 @@ void seat_client_destroy_touch(struct wl_resource *resource) {
 	}
 	wl_resource_set_user_data(resource, NULL);
 }
+
+bool wlr_seat_validate_touch_grab_serial(struct wlr_seat *seat,
+		struct wlr_surface *origin, uint32_t serial,
+		struct wlr_touch_point **point_ptr) {
+	if (wlr_seat_touch_num_points(seat) != 1 ||
+			seat->touch_state.grab_serial != serial) {
+		return false;
+	}
+
+	struct wlr_touch_point *point;
+	wl_list_for_each(point, &seat->touch_state.touch_points, link) {
+		if (origin == NULL || point->surface == origin) {
+			if (point_ptr != NULL) {
+				*point_ptr = point;
+			}
+			return true;
+		}
+	}
+
+	return false;
+}