Merge pull request #1427 from emersion/refactor-data-device

Refactor data-device
This commit is contained in:
Drew DeVault 2019-01-24 09:48:26 -05:00 committed by GitHub
commit 460a630a43
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
11 changed files with 173 additions and 122 deletions

View File

@ -16,8 +16,8 @@ struct wlr_client_data_source {
extern const struct wlr_surface_role drag_icon_surface_role; extern const struct wlr_surface_role drag_icon_surface_role;
struct wlr_data_offer *data_offer_create(struct wl_client *client, struct wlr_data_offer *data_offer_create(struct wl_resource *device_resource,
struct wlr_data_source *source, uint32_t version); struct wlr_data_source *source, enum wlr_data_offer_type type);
void data_offer_update_action(struct wlr_data_offer *offer); void data_offer_update_action(struct wlr_data_offer *offer);
void data_offer_destroy(struct wlr_data_offer *offer); void data_offer_destroy(struct wlr_data_offer *offer);
@ -26,12 +26,20 @@ struct wlr_client_data_source *client_data_source_create(
struct wl_list *resource_list); struct wl_list *resource_list);
struct wlr_client_data_source *client_data_source_from_resource( struct wlr_client_data_source *client_data_source_from_resource(
struct wl_resource *resource); struct wl_resource *resource);
struct wlr_data_offer *data_source_send_offer(struct wlr_data_source *source,
struct wl_resource *device_resource);
void data_source_notify_finish(struct wlr_data_source *source); 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, bool seat_client_start_drag(struct wlr_seat_client *client,
struct wlr_data_source *source, struct wlr_surface *icon_surface, struct wlr_data_source *source, struct wlr_surface *icon_surface,
struct wlr_surface *origin, uint32_t serial); 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
* wl_data_device.selection() event. If there is no current selection, the
* wl_data_device.selection() event will carry a NULL wl_data_offer. If the
* client does not have a wl_data_device for the seat nothing will be done.
*/
void seat_client_send_selection(struct wlr_seat_client *seat_client);
#endif #endif

View File

@ -35,9 +35,16 @@ struct wlr_data_device_manager {
void *data; void *data;
}; };
enum wlr_data_offer_type {
WLR_DATA_OFFER_SELECTION,
WLR_DATA_OFFER_DRAG,
};
struct wlr_data_offer { struct wlr_data_offer {
struct wl_resource *resource; struct wl_resource *resource;
struct wlr_data_source *source; struct wlr_data_source *source;
enum wlr_data_offer_type type;
struct wl_list link; // wlr_seat::{selection_offers,drag_offers}
uint32_t actions; uint32_t actions;
enum wl_data_device_manager_dnd_action preferred_action; enum wl_data_device_manager_dnd_action preferred_action;
@ -55,7 +62,7 @@ struct wlr_data_source_impl {
int32_t fd); int32_t fd);
void (*accept)(struct wlr_data_source *source, uint32_t serial, void (*accept)(struct wlr_data_source *source, uint32_t serial,
const char *mime_type); const char *mime_type);
void (*cancel)(struct wlr_data_source *source); void (*destroy)(struct wlr_data_source *source);
void (*dnd_drop)(struct wlr_data_source *source); void (*dnd_drop)(struct wlr_data_source *source);
void (*dnd_finish)(struct wlr_data_source *source); void (*dnd_finish)(struct wlr_data_source *source);
@ -156,15 +163,6 @@ struct wlr_data_device_manager *wlr_data_device_manager_create(
*/ */
void wlr_data_device_manager_destroy(struct wlr_data_device_manager *manager); void wlr_data_device_manager_destroy(struct wlr_data_device_manager *manager);
/**
* 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
* wl_data_device.selection() event. If there is no current selection, the
* wl_data_device.selection() event will carry a NULL wl_data_offer. If the
* client does not have a wl_data_device for the seat nothing * will be done.
*/
void wlr_seat_client_send_selection(struct wlr_seat_client *seat_client);
/** /**
* Requests a selection to be set for the seat. * Requests a selection to be set for the seat.
*/ */
@ -186,11 +184,6 @@ void wlr_seat_set_selection(struct wlr_seat *seat,
void wlr_data_source_init(struct wlr_data_source *source, void wlr_data_source_init(struct wlr_data_source *source,
const struct wlr_data_source_impl *impl); const struct wlr_data_source_impl *impl);
/**
* Finishes the data source.
*/
void wlr_data_source_finish(struct wlr_data_source *source);
/** /**
* Sends the data as the specified MIME type over the passed file descriptor, * Sends the data as the specified MIME type over the passed file descriptor,
* then close it. * then close it.
@ -207,9 +200,9 @@ void wlr_data_source_accept(struct wlr_data_source *source, uint32_t serial,
/** /**
* Notifies the data source it is no longer valid and should be destroyed. That * Notifies the data source it is no longer valid and should be destroyed. That
* potentially destroys immediately the data source. * destroys immediately the data source.
*/ */
void wlr_data_source_cancel(struct wlr_data_source *source); void wlr_data_source_destroy(struct wlr_data_source *source);
/** /**
* Notifies the data source that the drop operation was performed. This does not * Notifies the data source that the drop operation was performed. This does not

View File

@ -197,6 +197,7 @@ struct wlr_seat {
struct wlr_data_source *selection_source; struct wlr_data_source *selection_source;
uint32_t selection_serial; uint32_t selection_serial;
struct wl_list selection_offers; // wlr_data_offer::link
struct wlr_primary_selection_source *primary_selection_source; struct wlr_primary_selection_source *primary_selection_source;
uint32_t primary_selection_serial; uint32_t primary_selection_serial;
@ -205,6 +206,7 @@ struct wlr_seat {
struct wlr_drag *drag; struct wlr_drag *drag;
struct wlr_data_source *drag_source; struct wlr_data_source *drag_source;
uint32_t drag_serial; uint32_t drag_serial;
struct wl_list drag_offers; // wlr_data_offer::link
struct wlr_seat_pointer_state pointer_state; struct wlr_seat_pointer_state pointer_state;
struct wlr_seat_keyboard_state keyboard_state; struct wlr_seat_keyboard_state keyboard_state;

View File

@ -14,7 +14,7 @@
static const struct wl_data_device_interface data_device_impl; static const struct wl_data_device_interface data_device_impl;
static struct wlr_seat_client *seat_client_from_data_device_resource( struct wlr_seat_client *seat_client_from_data_device_resource(
struct wl_resource *resource) { struct wl_resource *resource) {
assert(wl_resource_instance_of(resource, &wl_data_device_interface, assert(wl_resource_instance_of(resource, &wl_data_device_interface,
&data_device_impl)); &data_device_impl));
@ -26,6 +26,9 @@ static void data_device_set_selection(struct wl_client *client,
struct wl_resource *source_resource, uint32_t serial) { struct wl_resource *source_resource, uint32_t serial) {
struct wlr_seat_client *seat_client = struct wlr_seat_client *seat_client =
seat_client_from_data_device_resource(device_resource); seat_client_from_data_device_resource(device_resource);
if (seat_client == NULL) {
return;
}
struct wlr_client_data_source *source = NULL; struct wlr_client_data_source *source = NULL;
if (source_resource != NULL) { if (source_resource != NULL) {
@ -48,6 +51,10 @@ static void data_device_start_drag(struct wl_client *client,
uint32_t serial) { uint32_t serial) {
struct wlr_seat_client *seat_client = struct wlr_seat_client *seat_client =
seat_client_from_data_device_resource(device_resource); seat_client_from_data_device_resource(device_resource);
if (seat_client == NULL) {
return;
}
struct wlr_surface *origin = wlr_surface_from_resource(origin_resource); struct wlr_surface *origin = wlr_surface_from_resource(origin_resource);
struct wlr_client_data_source *source = NULL; struct wlr_client_data_source *source = NULL;
@ -93,17 +100,15 @@ static void data_device_handle_resource_destroy(struct wl_resource *resource) {
} }
void wlr_seat_client_send_selection(struct wlr_seat_client *seat_client) { static void device_resource_send_selection(struct wl_resource *device_resource) {
struct wlr_seat_client *seat_client =
seat_client_from_data_device_resource(device_resource);
assert(seat_client != NULL);
struct wlr_data_source *source = seat_client->seat->selection_source; struct wlr_data_source *source = seat_client->seat->selection_source;
if (source != NULL) { if (source != NULL) {
source->accepted = false; struct wlr_data_offer *offer = data_offer_create(device_resource,
} source, WLR_DATA_OFFER_SELECTION);
struct wl_resource *device_resource;
wl_resource_for_each(device_resource, &seat_client->data_devices) {
if (source != NULL) {
struct wlr_data_offer *offer =
data_source_send_offer(source, device_resource);
if (offer == NULL) { if (offer == NULL) {
wl_client_post_no_memory(seat_client->client); wl_client_post_no_memory(seat_client->client);
return; return;
@ -114,6 +119,17 @@ void wlr_seat_client_send_selection(struct wlr_seat_client *seat_client) {
wl_data_device_send_selection(device_resource, NULL); wl_data_device_send_selection(device_resource, NULL);
} }
} }
void seat_client_send_selection(struct wlr_seat_client *seat_client) {
struct wlr_data_source *source = seat_client->seat->selection_source;
if (source != NULL) {
source->accepted = false;
}
struct wl_resource *device_resource;
wl_resource_for_each(device_resource, &seat_client->data_devices) {
device_resource_send_selection(device_resource);
}
} }
void wlr_seat_request_set_selection(struct wlr_seat *seat, void wlr_seat_request_set_selection(struct wlr_seat *seat,
@ -136,16 +152,14 @@ static void seat_handle_selection_source_destroy(
struct wl_listener *listener, void *data) { struct wl_listener *listener, void *data) {
struct wlr_seat *seat = struct wlr_seat *seat =
wl_container_of(listener, seat, selection_source_destroy); wl_container_of(listener, seat, selection_source_destroy);
struct wlr_seat_client *seat_client = seat->keyboard_state.focused_client;
wl_list_remove(&seat->selection_source_destroy.link); wl_list_remove(&seat->selection_source_destroy.link);
seat->selection_source = NULL; seat->selection_source = NULL;
if (seat_client && seat->keyboard_state.focused_surface) { struct wlr_seat_client *focused_client =
struct wl_resource *resource; seat->keyboard_state.focused_client;
wl_resource_for_each(resource, &seat_client->data_devices) { if (focused_client != NULL) {
wl_data_device_send_selection(resource, NULL); seat_client_send_selection(focused_client);
}
} }
wlr_signal_emit_safe(&seat->events.set_selection, seat); wlr_signal_emit_safe(&seat->events.set_selection, seat);
@ -155,19 +169,13 @@ void wlr_seat_set_selection(struct wlr_seat *seat,
struct wlr_data_source *source, uint32_t serial) { struct wlr_data_source *source, uint32_t serial) {
if (seat->selection_source) { if (seat->selection_source) {
wl_list_remove(&seat->selection_source_destroy.link); wl_list_remove(&seat->selection_source_destroy.link);
wlr_data_source_cancel(seat->selection_source); wlr_data_source_destroy(seat->selection_source);
seat->selection_source = NULL; seat->selection_source = NULL;
} }
seat->selection_source = source; seat->selection_source = source;
seat->selection_serial = serial; seat->selection_serial = serial;
struct wlr_seat_client *focused_client =
seat->keyboard_state.focused_client;
if (focused_client) {
wlr_seat_client_send_selection(focused_client);
}
if (source) { if (source) {
seat->selection_source_destroy.notify = seat->selection_source_destroy.notify =
seat_handle_selection_source_destroy; seat_handle_selection_source_destroy;
@ -175,6 +183,12 @@ void wlr_seat_set_selection(struct wlr_seat *seat,
&seat->selection_source_destroy); &seat->selection_source_destroy);
} }
struct wlr_seat_client *focused_client =
seat->keyboard_state.focused_client;
if (focused_client) {
seat_client_send_selection(focused_client);
}
wlr_signal_emit_safe(&seat->events.set_selection, seat); wlr_signal_emit_safe(&seat->events.set_selection, seat);
} }
@ -194,16 +208,22 @@ static void data_device_manager_get_data_device(struct wl_client *client,
struct wlr_seat_client *seat_client = struct wlr_seat_client *seat_client =
wlr_seat_client_from_resource(seat_resource); wlr_seat_client_from_resource(seat_resource);
uint32_t version = wl_resource_get_version(manager_resource);
struct wl_resource *resource = wl_resource_create(client, struct wl_resource *resource = wl_resource_create(client,
&wl_data_device_interface, wl_resource_get_version(manager_resource), &wl_data_device_interface, version, id);
id);
if (resource == NULL) { if (resource == NULL) {
wl_resource_post_no_memory(manager_resource); wl_resource_post_no_memory(manager_resource);
return; return;
} }
wl_resource_set_implementation(resource, &data_device_impl, seat_client, wl_resource_set_implementation(resource, &data_device_impl, seat_client,
&data_device_handle_resource_destroy); data_device_handle_resource_destroy);
wl_list_insert(&seat_client->data_devices, wl_resource_get_link(resource)); wl_list_insert(&seat_client->data_devices, wl_resource_get_link(resource));
struct wlr_seat_client *focused_client =
seat_client->seat->keyboard_state.focused_client;
if (focused_client == seat_client) {
device_resource_send_selection(resource);
}
} }
static void data_device_manager_create_data_source(struct wl_client *client, static void data_device_manager_create_data_source(struct wl_client *client,
@ -231,8 +251,7 @@ static void data_device_manager_bind(struct wl_client *client,
struct wlr_data_device_manager *manager = data; struct wlr_data_device_manager *manager = data;
struct wl_resource *resource = wl_resource_create(client, struct wl_resource *resource = wl_resource_create(client,
&wl_data_device_manager_interface, &wl_data_device_manager_interface, version, id);
version, id);
if (resource == NULL) { if (resource == NULL) {
wl_client_post_no_memory(client); wl_client_post_no_memory(client);
return; return;

View File

@ -55,6 +55,8 @@ static uint32_t data_offer_choose_action(struct wlr_data_offer *offer) {
} }
void data_offer_update_action(struct wlr_data_offer *offer) { void data_offer_update_action(struct wlr_data_offer *offer) {
assert(offer->type == WLR_DATA_OFFER_DRAG);
uint32_t action = data_offer_choose_action(offer); uint32_t action = data_offer_choose_action(offer);
if (offer->source->current_dnd_action == action) { if (offer->source->current_dnd_action == action) {
return; return;
@ -160,6 +162,13 @@ static void data_offer_handle_set_actions(struct wl_client *client,
return; return;
} }
if (offer->type != WLR_DATA_OFFER_DRAG) {
wl_resource_post_error(offer->resource,
WL_DATA_OFFER_ERROR_INVALID_OFFER,
"set_action can only be sent to drag-and-drop offers");
return;
}
offer->actions = actions; offer->actions = actions;
offer->preferred_action = preferred_action; offer->preferred_action = preferred_action;
@ -172,9 +181,11 @@ void data_offer_destroy(struct wlr_data_offer *offer) {
} }
wl_list_remove(&offer->source_destroy.link); wl_list_remove(&offer->source_destroy.link);
wl_list_remove(&offer->link);
// Make the resource inert // Make the resource inert
wl_resource_set_user_data(offer->resource, NULL); wl_resource_set_user_data(offer->resource, NULL);
free(offer); free(offer);
} }
@ -198,14 +209,22 @@ static void data_offer_handle_source_destroy(struct wl_listener *listener,
data_offer_destroy(offer); data_offer_destroy(offer);
} }
struct wlr_data_offer *data_offer_create(struct wl_client *client, struct wlr_data_offer *data_offer_create(struct wl_resource *device_resource,
struct wlr_data_source *source, uint32_t version) { struct wlr_data_source *source, enum wlr_data_offer_type type) {
struct wlr_seat_client *seat_client =
seat_client_from_data_device_resource(device_resource);
assert(seat_client != NULL);
assert(source != NULL); // a NULL source means no selection
struct wlr_data_offer *offer = calloc(1, sizeof(struct wlr_data_offer)); struct wlr_data_offer *offer = calloc(1, sizeof(struct wlr_data_offer));
if (offer == NULL) { if (offer == NULL) {
return NULL; return NULL;
} }
offer->source = source; offer->source = source;
offer->type = type;
struct wl_client *client = wl_resource_get_client(device_resource);
uint32_t version = wl_resource_get_version(device_resource);
offer->resource = offer->resource =
wl_resource_create(client, &wl_data_offer_interface, version, 0); wl_resource_create(client, &wl_data_offer_interface, version, 0);
if (offer->resource == NULL) { if (offer->resource == NULL) {
@ -215,8 +234,24 @@ struct wlr_data_offer *data_offer_create(struct wl_client *client,
wl_resource_set_implementation(offer->resource, &data_offer_impl, offer, wl_resource_set_implementation(offer->resource, &data_offer_impl, offer,
data_offer_handle_resource_destroy); data_offer_handle_resource_destroy);
switch (type) {
case WLR_DATA_OFFER_SELECTION:
wl_list_insert(&seat_client->seat->selection_offers, &offer->link);
break;
case WLR_DATA_OFFER_DRAG:
wl_list_insert(&seat_client->seat->drag_offers, &offer->link);
break;
}
offer->source_destroy.notify = data_offer_handle_source_destroy; offer->source_destroy.notify = data_offer_handle_source_destroy;
wl_signal_add(&source->events.destroy, &offer->source_destroy); wl_signal_add(&source->events.destroy, &offer->source_destroy);
wl_data_device_send_data_offer(device_resource, offer->resource);
char **p;
wl_array_for_each(p, &source->mime_types) {
wl_data_offer_send_offer(offer->resource, *p);
}
return offer; return offer;
} }

View File

@ -11,25 +11,6 @@
#include "types/wlr_data_device.h" #include "types/wlr_data_device.h"
#include "util/signal.h" #include "util/signal.h"
struct wlr_data_offer *data_source_send_offer(struct wlr_data_source *source,
struct wl_resource *device_resource) {
struct wl_client *client = wl_resource_get_client(device_resource);
uint32_t version = wl_resource_get_version(device_resource);
struct wlr_data_offer *offer = data_offer_create(client, source, version);
if (offer == NULL) {
return NULL;
}
wl_data_device_send_data_offer(device_resource, offer->resource);
char **p;
wl_array_for_each(p, &source->mime_types) {
wl_data_offer_send_offer(offer->resource, *p);
}
return offer;
}
void wlr_data_source_init(struct wlr_data_source *source, void wlr_data_source_init(struct wlr_data_source *source,
const struct wlr_data_source_impl *impl) { const struct wlr_data_source_impl *impl) {
assert(impl->send); assert(impl->send);
@ -40,20 +21,6 @@ void wlr_data_source_init(struct wlr_data_source *source,
source->actions = -1; source->actions = -1;
} }
void wlr_data_source_finish(struct wlr_data_source *source) {
if (source == NULL) {
return;
}
wlr_signal_emit_safe(&source->events.destroy, source);
char **p;
wl_array_for_each(p, &source->mime_types) {
free(*p);
}
wl_array_release(&source->mime_types);
}
void wlr_data_source_send(struct wlr_data_source *source, const char *mime_type, void wlr_data_source_send(struct wlr_data_source *source, const char *mime_type,
int32_t fd) { int32_t fd) {
source->impl->send(source, mime_type, fd); source->impl->send(source, mime_type, fd);
@ -67,9 +34,23 @@ void wlr_data_source_accept(struct wlr_data_source *source, uint32_t serial,
} }
} }
void wlr_data_source_cancel(struct wlr_data_source *source) { void wlr_data_source_destroy(struct wlr_data_source *source) {
if (source->impl->cancel) { if (source == NULL) {
source->impl->cancel(source); return;
}
wlr_signal_emit_safe(&source->events.destroy, source);
char **p;
wl_array_for_each(p, &source->mime_types) {
free(*p);
}
wl_array_release(&source->mime_types);
if (source->impl->destroy) {
source->impl->destroy(source);
} else {
free(source);
} }
} }
@ -127,10 +108,12 @@ static void client_data_source_send(struct wlr_data_source *wlr_source,
close(fd); close(fd);
} }
static void client_data_source_cancel(struct wlr_data_source *wlr_source) { static void client_data_source_destroy(struct wlr_data_source *wlr_source) {
struct wlr_client_data_source *source = struct wlr_client_data_source *source =
client_data_source_from_wlr_data_source(wlr_source); client_data_source_from_wlr_data_source(wlr_source);
wl_data_source_send_cancelled(source->resource); wl_data_source_send_cancelled(source->resource);
wl_resource_set_user_data(source->resource, NULL);
free(source);
} }
static void client_data_source_dnd_drop(struct wlr_data_source *wlr_source) { static void client_data_source_dnd_drop(struct wlr_data_source *wlr_source) {
@ -167,6 +150,9 @@ static void data_source_set_actions(struct wl_client *client,
struct wl_resource *resource, uint32_t dnd_actions) { struct wl_resource *resource, uint32_t dnd_actions) {
struct wlr_client_data_source *source = struct wlr_client_data_source *source =
client_data_source_from_resource(resource); client_data_source_from_resource(resource);
if (source == NULL) {
return;
}
if (source->source.actions >= 0) { if (source->source.actions >= 0) {
wl_resource_post_error(source->resource, wl_resource_post_error(source->resource,
@ -196,6 +182,13 @@ static void data_source_offer(struct wl_client *client,
struct wl_resource *resource, const char *mime_type) { struct wl_resource *resource, const char *mime_type) {
struct wlr_client_data_source *source = struct wlr_client_data_source *source =
client_data_source_from_resource(resource); client_data_source_from_resource(resource);
if (source == NULL) {
return;
}
if (source->finalized) {
wlr_log(WLR_DEBUG, "Offering additional MIME type after "
"wl_data_device.set_selection");
}
char **p = wl_array_add(&source->source.mime_types, sizeof(*p)); char **p = wl_array_add(&source->source.mime_types, sizeof(*p));
if (p) { if (p) {
@ -210,17 +203,18 @@ static void data_source_offer(struct wl_client *client,
} }
static const struct wl_data_source_interface data_source_impl = { static const struct wl_data_source_interface data_source_impl = {
.offer = data_source_offer,
.destroy = data_source_destroy, .destroy = data_source_destroy,
.offer = data_source_offer,
.set_actions = data_source_set_actions, .set_actions = data_source_set_actions,
}; };
static void data_source_handle_resource_destroy(struct wl_resource *resource) { static void data_source_handle_resource_destroy(struct wl_resource *resource) {
struct wlr_client_data_source *source = struct wlr_client_data_source *source =
client_data_source_from_resource(resource); client_data_source_from_resource(resource);
wlr_data_source_finish(&source->source); if (source != NULL) {
wl_list_remove(wl_resource_get_link(source->resource)); wlr_data_source_destroy(&source->source);
free(source); }
wl_list_remove(wl_resource_get_link(resource));
} }
struct wlr_client_data_source *client_data_source_create( struct wlr_client_data_source *client_data_source_create(
@ -245,7 +239,7 @@ struct wlr_client_data_source *client_data_source_create(
source->impl.accept = client_data_source_accept; source->impl.accept = client_data_source_accept;
source->impl.send = client_data_source_send; source->impl.send = client_data_source_send;
source->impl.cancel = client_data_source_cancel; source->impl.destroy = client_data_source_destroy;
if (wl_resource_get_version(source->resource) >= if (wl_resource_get_version(source->resource) >=
WL_DATA_SOURCE_DND_DROP_PERFORMED_SINCE_VERSION) { WL_DATA_SOURCE_DND_DROP_PERFORMED_SINCE_VERSION) {

View File

@ -61,8 +61,8 @@ static void drag_set_focus(struct wlr_drag *drag,
struct wl_resource *device_resource; struct wl_resource *device_resource;
wl_resource_for_each(device_resource, &focus_client->data_devices) { wl_resource_for_each(device_resource, &focus_client->data_devices) {
struct wlr_data_offer *offer = struct wlr_data_offer *offer = data_offer_create(device_resource,
data_source_send_offer(drag->source, device_resource); drag->source, WLR_DATA_OFFER_DRAG);
if (offer == NULL) { if (offer == NULL) {
wl_resource_post_no_memory(device_resource); wl_resource_post_no_memory(device_resource);
return; return;
@ -174,7 +174,7 @@ static uint32_t drag_handle_pointer_button(struct wlr_seat_pointer_grab *grab,
}; };
wlr_signal_emit_safe(&drag->events.drop, &event); wlr_signal_emit_safe(&drag->events.drop, &event);
} else if (drag->source->impl->dnd_finish) { } else if (drag->source->impl->dnd_finish) {
wlr_data_source_cancel(drag->source); wlr_data_source_destroy(drag->source);
} }
} }

View File

@ -86,7 +86,12 @@ static void seat_client_handle_resource_destroy(
wl_resource_destroy(resource); wl_resource_destroy(resource);
} }
wl_resource_for_each_safe(resource, tmp, &client->data_devices) { wl_resource_for_each_safe(resource, tmp, &client->data_devices) {
wl_resource_destroy(resource); // Make the data device inert
wl_resource_set_user_data(resource, NULL);
struct wl_list *link = wl_resource_get_link(resource);
wl_list_remove(link);
wl_list_init(link);
} }
wl_list_remove(&client->link); wl_list_remove(&client->link);
@ -157,14 +162,8 @@ void wlr_seat_destroy(struct wlr_seat *seat) {
wl_list_remove(&seat->display_destroy.link); wl_list_remove(&seat->display_destroy.link);
if (seat->selection_source) { wlr_data_source_destroy(seat->selection_source);
wl_list_remove(&seat->selection_source_destroy.link); wlr_primary_selection_source_destroy(seat->primary_selection_source);
wlr_data_source_cancel(seat->selection_source);
seat->selection_source = NULL;
}
wlr_seat_set_primary_selection(seat, NULL,
wl_display_next_serial(seat->display));
struct wlr_seat_client *client, *tmp; struct wlr_seat_client *client, *tmp;
wl_list_for_each_safe(client, tmp, &seat->clients, link) { wl_list_for_each_safe(client, tmp, &seat->clients, link) {
@ -268,6 +267,8 @@ struct wlr_seat *wlr_seat_create(struct wl_display *display, const char *name) {
seat->name = strdup(name); seat->name = strdup(name);
wl_list_init(&seat->clients); wl_list_init(&seat->clients);
wl_list_init(&seat->drag_icons); wl_list_init(&seat->drag_icons);
wl_list_init(&seat->selection_offers);
wl_list_init(&seat->drag_offers);
wl_signal_init(&seat->events.start_drag); wl_signal_init(&seat->events.start_drag);
wl_signal_init(&seat->events.new_drag_icon); wl_signal_init(&seat->events.new_drag_icon);

View File

@ -2,17 +2,18 @@
#include <assert.h> #include <assert.h>
#include <stdlib.h> #include <stdlib.h>
#include <string.h> #include <string.h>
#include <time.h>
#include <sys/mman.h> #include <sys/mman.h>
#include <time.h>
#include <unistd.h> #include <unistd.h>
#include <wayland-server.h> #include <wayland-server.h>
#include <wlr/types/wlr_data_device.h> #include <wlr/types/wlr_data_device.h>
#include <wlr/types/wlr_input_device.h>
#include <wlr/types/wlr_gtk_primary_selection.h> #include <wlr/types/wlr_gtk_primary_selection.h>
#include <wlr/types/wlr_input_device.h>
#include <wlr/util/log.h> #include <wlr/util/log.h>
#include "types/wlr_data_device.h"
#include "types/wlr_seat.h" #include "types/wlr_seat.h"
#include "util/signal.h"
#include "util/shm.h" #include "util/shm.h"
#include "util/signal.h"
static void default_keyboard_enter(struct wlr_seat_keyboard_grab *grab, static void default_keyboard_enter(struct wlr_seat_keyboard_grab *grab,
struct wlr_surface *surface, uint32_t keycodes[], size_t num_keycodes, struct wlr_surface *surface, uint32_t keycodes[], size_t num_keycodes,
@ -271,8 +272,6 @@ void wlr_seat_keyboard_enter(struct wlr_seat *seat,
wl_keyboard_send_enter(resource, serial, surface->resource, &keys); wl_keyboard_send_enter(resource, serial, surface->resource, &keys);
} }
wl_array_release(&keys); wl_array_release(&keys);
wlr_seat_client_send_selection(client);
} }
// reinitialize the focus destroy events // reinitialize the focus destroy events
@ -292,6 +291,8 @@ void wlr_seat_keyboard_enter(struct wlr_seat *seat,
// tell new client about any modifier change last, // tell new client about any modifier change last,
// as it targets seat->keyboard_state.focused_client // as it targets seat->keyboard_state.focused_client
wlr_seat_keyboard_send_modifiers(seat, modifiers); wlr_seat_keyboard_send_modifiers(seat, modifiers);
seat_client_send_selection(client);
} }
struct wlr_seat_keyboard_focus_change_event event = { struct wlr_seat_keyboard_focus_change_event event = {

View File

@ -32,11 +32,10 @@ static void client_source_send(struct wlr_data_source *wlr_source,
close(fd); close(fd);
} }
static void client_source_cancel(struct wlr_data_source *wlr_source) { static void client_source_destroy(struct wlr_data_source *wlr_source) {
struct client_data_source *source = struct client_data_source *source =
client_data_source_from_source(wlr_source); client_data_source_from_source(wlr_source);
zwlr_data_control_source_v1_send_cancelled(source->resource); zwlr_data_control_source_v1_send_cancelled(source->resource);
wlr_data_source_finish(wlr_source);
// Make the resource inert // Make the resource inert
wl_resource_set_user_data(source->resource, NULL); wl_resource_set_user_data(source->resource, NULL);
free(source); free(source);
@ -44,7 +43,7 @@ static void client_source_cancel(struct wlr_data_source *wlr_source) {
static const struct wlr_data_source_impl client_source_impl = { static const struct wlr_data_source_impl client_source_impl = {
.send = client_source_send, .send = client_source_send,
.cancel = client_source_cancel, .destroy = client_source_destroy,
}; };
static const struct zwlr_data_control_source_v1_interface source_impl; static const struct zwlr_data_control_source_v1_interface source_impl;
@ -101,7 +100,7 @@ static void source_handle_resource_destroy(struct wl_resource *resource) {
if (source == NULL) { if (source == NULL) {
return; return;
} }
wlr_data_source_cancel(&source->source); wlr_data_source_destroy(&source->source);
} }

View File

@ -203,17 +203,16 @@ static void data_source_send(struct wlr_data_source *wlr_source,
mime_type, fd); mime_type, fd);
} }
static void data_source_cancel(struct wlr_data_source *wlr_source) { static void data_source_destroy(struct wlr_data_source *wlr_source) {
struct x11_data_source *source = struct x11_data_source *source =
data_source_from_wlr_data_source(wlr_source); data_source_from_wlr_data_source(wlr_source);
wlr_data_source_finish(&source->base);
wl_array_release(&source->mime_types_atoms); wl_array_release(&source->mime_types_atoms);
free(source); free(source);
} }
static const struct wlr_data_source_impl data_source_impl = { static const struct wlr_data_source_impl data_source_impl = {
.send = data_source_send, .send = data_source_send,
.cancel = data_source_cancel, .destroy = data_source_destroy,
}; };
struct x11_primary_selection_source { struct x11_primary_selection_source {
@ -353,7 +352,7 @@ static void xwm_selection_get_targets(struct wlr_xwm_selection *selection) {
wlr_seat_request_set_selection(xwm->seat, &source->base, wlr_seat_request_set_selection(xwm->seat, &source->base,
wl_display_next_serial(xwm->xwayland->wl_display)); wl_display_next_serial(xwm->xwayland->wl_display));
} else { } else {
wlr_data_source_cancel(&source->base); wlr_data_source_destroy(&source->base);
} }
} else if (selection == &xwm->primary_selection) { } else if (selection == &xwm->primary_selection) {
struct x11_primary_selection_source *source = struct x11_primary_selection_source *source =