Make wlr_data_source abstract.

This removes some fields specific to sources coming from clients.

This adds some drag'n'drop-related callbacks.
This commit is contained in:
emersion 2017-12-29 22:51:33 +01:00
parent 80ed4d4d20
commit f237b5c7a7
No known key found for this signature in database
GPG key ID: 0FDE7BE0E88F5E48
3 changed files with 149 additions and 101 deletions

View file

@ -23,33 +23,39 @@ struct wlr_data_offer {
struct wl_resource *resource; struct wl_resource *resource;
struct wlr_data_source *source; struct wlr_data_source *source;
uint32_t dnd_actions; uint32_t actions;
enum wl_data_device_manager_dnd_action preferred_dnd_action; enum wl_data_device_manager_dnd_action preferred_action;
bool in_ask; bool in_ask;
struct wl_listener source_destroy; struct wl_listener source_destroy;
}; };
struct wlr_data_source { struct wlr_data_source {
struct wl_resource *resource; // source metadata
struct wl_array mime_types;
int32_t actions;
// source implementation
void (*send)(struct wlr_data_source *source, const char *mime_type,
int32_t fd);
void (*accept)(struct wlr_data_source *source, uint32_t serial,
const char *mime_type);
void (*cancel)(struct wlr_data_source *source);
// drag'n'drop implementation
void (*dnd_drop)(struct wlr_data_source *source);
void (*dnd_finish)(struct wlr_data_source *source);
void (*dnd_action)(struct wlr_data_source *source,
enum wl_data_device_manager_dnd_action action);
// source status
bool accepted;
struct wlr_data_offer *offer; struct wlr_data_offer *offer;
struct wlr_seat_client *seat_client; struct wlr_seat_client *seat_client;
struct wl_array mime_types; // drag'n'drop status
bool accepted;
// drag and drop
enum wl_data_device_manager_dnd_action current_dnd_action; enum wl_data_device_manager_dnd_action current_dnd_action;
uint32_t dnd_actions;
uint32_t compositor_action; uint32_t compositor_action;
bool actions_set;
void (*accept)(struct wlr_data_source *source, uint32_t serial,
const char *mime_type);
void (*send)(struct wlr_data_source *source, const char *mime_type,
int32_t fd);
void (*cancel)(struct wlr_data_source *source);
struct { struct {
struct wl_signal destroy; struct wl_signal destroy;

View file

@ -14,26 +14,23 @@
WL_DATA_DEVICE_MANAGER_DND_ACTION_ASK) WL_DATA_DEVICE_MANAGER_DND_ACTION_ASK)
static uint32_t data_offer_choose_action(struct wlr_data_offer *offer) { static uint32_t data_offer_choose_action(struct wlr_data_offer *offer) {
uint32_t available_actions, preferred_action = 0; uint32_t offer_actions, preferred_action = 0;
uint32_t source_actions, offer_actions;
if (wl_resource_get_version(offer->resource) >= if (wl_resource_get_version(offer->resource) >=
WL_DATA_OFFER_ACTION_SINCE_VERSION) { WL_DATA_OFFER_ACTION_SINCE_VERSION) {
offer_actions = offer->dnd_actions; offer_actions = offer->actions;
preferred_action = offer->preferred_dnd_action; preferred_action = offer->preferred_action;
} else { } else {
offer_actions = WL_DATA_DEVICE_MANAGER_DND_ACTION_COPY; offer_actions = WL_DATA_DEVICE_MANAGER_DND_ACTION_COPY;
} }
if (wl_resource_get_version(offer->source->resource) >= uint32_t source_actions;
WL_DATA_SOURCE_ACTION_SINCE_VERSION) { if (offer->source->actions >= 0) {
source_actions = offer->source->dnd_actions; source_actions = offer->source->actions;
} else { } else {
source_actions = WL_DATA_DEVICE_MANAGER_DND_ACTION_COPY; source_actions = WL_DATA_DEVICE_MANAGER_DND_ACTION_COPY;
} }
available_actions = offer_actions & source_actions; uint32_t available_actions = offer_actions & source_actions;
if (!available_actions) { if (!available_actions) {
return WL_DATA_DEVICE_MANAGER_DND_ACTION_NONE; return WL_DATA_DEVICE_MANAGER_DND_ACTION_NONE;
} }
@ -53,14 +50,11 @@ static uint32_t data_offer_choose_action(struct wlr_data_offer *offer) {
} }
static void data_offer_update_action(struct wlr_data_offer *offer) { static void data_offer_update_action(struct wlr_data_offer *offer) {
uint32_t action;
if (!offer->source) { if (!offer->source) {
return; return;
} }
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;
} }
@ -71,9 +65,8 @@ static void data_offer_update_action(struct wlr_data_offer *offer) {
return; return;
} }
if (wl_resource_get_version(offer->source->resource) >= if (offer->source->dnd_action) {
WL_DATA_SOURCE_ACTION_SINCE_VERSION) { offer->source->dnd_action(offer->source, action);
wl_data_source_send_action(offer->source->resource, action);
} }
if (wl_resource_get_version(offer->resource) >= if (wl_resource_get_version(offer->resource) >=
@ -82,21 +75,58 @@ static void data_offer_update_action(struct wlr_data_offer *offer) {
} }
} }
static void client_data_source_accept(struct wlr_data_source *source, struct client_data_source {
struct wlr_data_source source;
struct wl_resource *resource;
bool actions_set;
};
static void client_data_source_accept(struct wlr_data_source *wlr_source,
uint32_t serial, const char *mime_type) { uint32_t serial, const char *mime_type) {
struct client_data_source *source = (struct client_data_source *)wlr_source;
wl_data_source_send_target(source->resource, mime_type); wl_data_source_send_target(source->resource, mime_type);
} }
static void client_data_source_send(struct wlr_data_source *source, static void client_data_source_send(struct wlr_data_source *wlr_source,
const char *mime_type, int32_t fd) { const char *mime_type, int32_t fd) {
struct client_data_source *source = (struct client_data_source *)wlr_source;
wl_data_source_send_send(source->resource, mime_type, fd); wl_data_source_send_send(source->resource, mime_type, fd);
close(fd); close(fd);
} }
static void client_data_source_cancel(struct wlr_data_source *source) { static void client_data_source_cancel(struct wlr_data_source *wlr_source) {
struct client_data_source *source = (struct client_data_source *)wlr_source;
wl_data_source_send_cancelled(source->resource); wl_data_source_send_cancelled(source->resource);
} }
static void client_data_source_dnd_drop(struct wlr_data_source *wlr_source) {
struct client_data_source *source = (struct client_data_source *)wlr_source;
assert(wl_resource_get_version(source->resource) >=
WL_DATA_SOURCE_DND_DROP_PERFORMED_SINCE_VERSION);
wl_data_source_send_dnd_drop_performed(source->resource);
}
static void client_data_source_dnd_finish(struct wlr_data_source *wlr_source) {
struct client_data_source *source = (struct client_data_source *)wlr_source;
assert(wl_resource_get_version(source->resource) >=
WL_DATA_SOURCE_DND_FINISHED_SINCE_VERSION);
wl_data_source_send_dnd_finished(source->resource);
}
static void client_data_source_dnd_action(struct wlr_data_source *wlr_source,
enum wl_data_device_manager_dnd_action action) {
struct client_data_source *source = (struct client_data_source *)wlr_source;
assert(wl_resource_get_version(source->resource) >=
WL_DATA_SOURCE_ACTION_SINCE_VERSION);
wl_data_source_send_action(source->resource, action);
}
static void client_data_source_resource_destroy(struct wl_resource *resource) {
struct client_data_source *source = wl_resource_get_user_data(resource);
wlr_data_source_finish(&source->source);
free(source);
}
static void data_offer_accept(struct wl_client *client, static void data_offer_accept(struct wl_client *client,
struct wl_resource *resource, uint32_t serial, const char *mime_type) { struct wl_resource *resource, uint32_t serial, const char *mime_type) {
@ -108,7 +138,9 @@ static void data_offer_accept(struct wl_client *client,
// TODO check that client is currently focused by the input device // TODO check that client is currently focused by the input device
if (offer->source->accept) {
offer->source->accept(offer->source, serial, mime_type); offer->source->accept(offer->source, serial, mime_type);
}
offer->source->accepted = (mime_type != NULL); offer->source->accepted = (mime_type != NULL);
} }
@ -128,19 +160,17 @@ static void data_offer_destroy(struct wl_client *client,
} }
static void data_source_notify_finish(struct wlr_data_source *source) { static void data_source_notify_finish(struct wlr_data_source *source) {
if (!source->actions_set) { assert(source->offer);
if (source->actions < 0) {
return; return;
} }
if (source->offer->in_ask && wl_resource_get_version(source->resource) >= if (source->offer->in_ask && source->dnd_action) {
WL_DATA_SOURCE_ACTION_SINCE_VERSION) { source->dnd_action(source, source->current_dnd_action);
wl_data_source_send_action(source->resource,
source->current_dnd_action);
} }
if (wl_resource_get_version(source->resource) >= if (source->dnd_finish) {
WL_DATA_SOURCE_DND_FINISHED_SINCE_VERSION) { source->dnd_finish(source);
wl_data_source_send_dnd_finished(source->resource);
} }
source->offer = NULL; source->offer = NULL;
@ -158,18 +188,18 @@ static void data_offer_finish(struct wl_client *client,
} }
static void data_offer_set_actions(struct wl_client *client, static void data_offer_set_actions(struct wl_client *client,
struct wl_resource *resource, uint32_t dnd_actions, struct wl_resource *resource, uint32_t actions,
uint32_t preferred_action) { uint32_t preferred_action) {
struct wlr_data_offer *offer = wl_resource_get_user_data(resource); struct wlr_data_offer *offer = wl_resource_get_user_data(resource);
if (dnd_actions & ~ALL_ACTIONS) { if (actions & ~ALL_ACTIONS) {
wl_resource_post_error(offer->resource, wl_resource_post_error(offer->resource,
WL_DATA_OFFER_ERROR_INVALID_ACTION_MASK, WL_DATA_OFFER_ERROR_INVALID_ACTION_MASK,
"invalid action mask %x", dnd_actions); "invalid action mask %x", actions);
return; return;
} }
if (preferred_action && (!(preferred_action & dnd_actions) || if (preferred_action && (!(preferred_action & actions) ||
__builtin_popcount(preferred_action) > 1)) { __builtin_popcount(preferred_action) > 1)) {
wl_resource_post_error(offer->resource, wl_resource_post_error(offer->resource,
WL_DATA_OFFER_ERROR_INVALID_ACTION, WL_DATA_OFFER_ERROR_INVALID_ACTION,
@ -177,8 +207,8 @@ static void data_offer_set_actions(struct wl_client *client,
return; return;
} }
offer->dnd_actions = dnd_actions; offer->actions = actions;
offer->preferred_dnd_action = preferred_action; offer->preferred_action = preferred_action;
data_offer_update_action(offer); data_offer_update_action(offer);
} }
@ -198,14 +228,12 @@ static void data_offer_resource_destroy(struct wl_resource *resource) {
// If the drag destination has version < 3, wl_data_offer.finish // If the drag destination has version < 3, wl_data_offer.finish
// won't be called, so do this here as a safety net, because // won't be called, so do this here as a safety net, because
// we still want the version >=3 drag source to be happy. // we still want the version >= 3 drag source to be happy.
if (wl_resource_get_version(offer->resource) < if (wl_resource_get_version(offer->resource) <
WL_DATA_OFFER_ACTION_SINCE_VERSION) { WL_DATA_OFFER_ACTION_SINCE_VERSION) {
data_source_notify_finish(offer->source); data_source_notify_finish(offer->source);
} else if (offer->source->resource && } else if (offer->source->dnd_finish) {
wl_resource_get_version(offer->source->resource) >= offer->source->cancel(offer->source);
WL_DATA_SOURCE_DND_FINISHED_SINCE_VERSION) {
wl_data_source_send_cancelled(offer->source->resource);
} }
offer->source->offer = NULL; offer->source->offer = NULL;
@ -316,6 +344,8 @@ static void seat_client_selection_data_source_destroy(
void wlr_seat_set_selection(struct wlr_seat *seat, 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) {
assert(source->send);
assert(source->cancel);
if (seat->selection_source && if (seat->selection_source &&
seat->selection_serial - serial < UINT32_MAX / 2) { seat->selection_serial - serial < UINT32_MAX / 2) {
return; return;
@ -433,7 +463,7 @@ static void wlr_drag_set_focus(struct wlr_drag *drag,
if (wl_resource_get_version(offer->resource) >= if (wl_resource_get_version(offer->resource) >=
WL_DATA_OFFER_SOURCE_ACTIONS_SINCE_VERSION) { WL_DATA_OFFER_SOURCE_ACTIONS_SINCE_VERSION) {
wl_data_offer_send_source_actions(offer->resource, wl_data_offer_send_source_actions(offer->resource,
drag->source->dnd_actions); drag->source->actions);
} }
offer_resource = offer->resource; offer_resource = offer->resource;
@ -510,18 +540,15 @@ static uint32_t pointer_drag_button(struct wlr_seat_pointer_grab *grab,
wl_resource_for_each(resource, &drag->focus_client->data_devices) { wl_resource_for_each(resource, &drag->focus_client->data_devices) {
wl_data_device_send_drop(resource); wl_data_device_send_drop(resource);
} }
if (wl_resource_get_version(drag->source->resource) >= if (drag->source->dnd_drop) {
WL_DATA_SOURCE_DND_DROP_PERFORMED_SINCE_VERSION) { drag->source->dnd_drop(drag->source);
wl_data_source_send_dnd_drop_performed(
drag->source->resource);
} }
drag->source->offer->in_ask = drag->source->offer->in_ask =
drag->source->current_dnd_action == drag->source->current_dnd_action ==
WL_DATA_DEVICE_MANAGER_DND_ACTION_ASK; WL_DATA_DEVICE_MANAGER_DND_ACTION_ASK;
} else if (wl_resource_get_version(drag->source->resource) >= } else if (drag->source->dnd_finish) {
WL_DATA_SOURCE_DND_FINISHED_SINCE_VERSION) { drag->source->cancel(drag->source);
wl_data_source_send_cancelled(drag->source->resource);
} }
} }
@ -852,13 +879,6 @@ void data_device_manager_get_data_device(struct wl_client *client,
wl_list_insert(&seat_client->data_devices, wl_resource_get_link(resource)); wl_list_insert(&seat_client->data_devices, wl_resource_get_link(resource));
} }
static void data_source_resource_destroy(struct wl_resource *resource) {
struct wlr_data_source *source =
wl_resource_get_user_data(resource);
wlr_data_source_finish(source);
free(source);
}
static void data_source_destroy(struct wl_client *client, static void data_source_destroy(struct wl_client *client,
struct wl_resource *resource) { struct wl_resource *resource) {
wl_resource_destroy(resource); wl_resource_destroy(resource);
@ -866,10 +886,10 @@ static void data_source_destroy(struct wl_client *client,
static void data_source_set_actions(struct wl_client *client, 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_data_source *source = struct client_data_source *source =
wl_resource_get_user_data(resource); wl_resource_get_user_data(resource);
if (source->actions_set) { if (source->source.actions >= 0) {
wl_resource_post_error(source->resource, wl_resource_post_error(source->resource,
WL_DATA_SOURCE_ERROR_INVALID_ACTION_MASK, WL_DATA_SOURCE_ERROR_INVALID_ACTION_MASK,
"cannot set actions more than once"); "cannot set actions more than once");
@ -883,7 +903,7 @@ static void data_source_set_actions(struct wl_client *client,
return; return;
} }
if (source->seat_client) { if (source->source.seat_client) {
wl_resource_post_error(source->resource, wl_resource_post_error(source->resource,
WL_DATA_SOURCE_ERROR_INVALID_ACTION_MASK, WL_DATA_SOURCE_ERROR_INVALID_ACTION_MASK,
"invalid action change after " "invalid action change after "
@ -891,22 +911,18 @@ static void data_source_set_actions(struct wl_client *client,
return; return;
} }
source->dnd_actions = dnd_actions; source->source.actions = dnd_actions;
source->actions_set = true;
} }
static void data_source_offer(struct wl_client *client, 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_data_source *source = struct wlr_data_source *source = wl_resource_get_user_data(resource);
wl_resource_get_user_data(resource);
char **p;
p = wl_array_add(&source->mime_types, sizeof(*p));
char **p = wl_array_add(&source->mime_types, sizeof(*p));
if (p) { if (p) {
*p = strdup(mime_type); *p = strdup(mime_type);
} }
if (!p || !*p){ if (!p || !*p) {
if (p) { if (p) {
source->mime_types.size -= sizeof(*p); source->mime_types.size -= sizeof(*p);
} }
@ -923,6 +939,7 @@ static struct wl_data_source_interface data_source_impl = {
void wlr_data_source_init(struct wlr_data_source *source) { void wlr_data_source_init(struct wlr_data_source *source) {
wl_array_init(&source->mime_types); wl_array_init(&source->mime_types);
wl_signal_init(&source->events.destroy); wl_signal_init(&source->events.destroy);
source->actions = -1;
} }
void wlr_data_source_finish(struct wlr_data_source *source) { void wlr_data_source_finish(struct wlr_data_source *source) {
@ -941,12 +958,13 @@ void wlr_data_source_finish(struct wlr_data_source *source) {
static void data_device_manager_create_data_source(struct wl_client *client, static void data_device_manager_create_data_source(struct wl_client *client,
struct wl_resource *resource, uint32_t id) { struct wl_resource *resource, uint32_t id) {
struct wlr_data_source *source = calloc(1, sizeof(struct wlr_data_source)); struct client_data_source *source =
calloc(1, sizeof(struct client_data_source));
if (source == NULL) { if (source == NULL) {
wl_resource_post_no_memory(resource); wl_resource_post_no_memory(resource);
return; return;
} }
wlr_data_source_init(source); wlr_data_source_init(&source->source);
source->resource = wl_resource_create(client, &wl_data_source_interface, source->resource = wl_resource_create(client, &wl_data_source_interface,
wl_resource_get_version(resource), id); wl_resource_get_version(resource), id);
@ -956,11 +974,24 @@ static void data_device_manager_create_data_source(struct wl_client *client,
return; return;
} }
wl_resource_set_implementation(source->resource, &data_source_impl, wl_resource_set_implementation(source->resource, &data_source_impl,
source, data_source_resource_destroy); source, client_data_source_resource_destroy);
source->accept = client_data_source_accept; source->source.accept = client_data_source_accept;
source->send = client_data_source_send; source->source.send = client_data_source_send;
source->cancel = client_data_source_cancel; source->source.cancel = client_data_source_cancel;
if (wl_resource_get_version(source->resource) >=
WL_DATA_SOURCE_DND_DROP_PERFORMED_SINCE_VERSION) {
source->source.dnd_drop = client_data_source_dnd_drop;
}
if (wl_resource_get_version(source->resource) >=
WL_DATA_SOURCE_DND_FINISHED_SINCE_VERSION) {
source->source.dnd_finish = client_data_source_dnd_finish;
}
if (wl_resource_get_version(source->resource) >=
WL_DATA_SOURCE_ACTION_SINCE_VERSION) {
source->source.dnd_action = client_data_source_dnd_action;
}
} }
static const struct wl_data_device_manager_interface static const struct wl_data_device_manager_interface

View file

@ -343,28 +343,39 @@ static const struct wlr_touch_grab_interface default_touch_grab_impl = {
}; };
void wlr_seat_destroy(struct wlr_seat *wlr_seat) { void wlr_seat_destroy(struct wlr_seat *seat) {
if (!wlr_seat) { if (!seat) {
return; return;
} }
wl_signal_emit(&wlr_seat->events.destroy, wlr_seat); wl_signal_emit(&seat->events.destroy, seat);
wl_list_remove(&wlr_seat->display_destroy.link); wl_list_remove(&seat->display_destroy.link);
if (seat->selection_source) {
seat->selection_source->cancel(seat->selection_source);
seat->selection_source = NULL;
wl_list_remove(&seat->selection_data_source_destroy.link);
}
if (seat->primary_selection_source) {
seat->primary_selection_source->cancel(seat->primary_selection_source);
seat->primary_selection_source = NULL;
wl_list_remove(&seat->primary_selection_source_destroy.link);
}
struct wlr_seat_client *client, *tmp; struct wlr_seat_client *client, *tmp;
wl_list_for_each_safe(client, tmp, &wlr_seat->clients, link) { wl_list_for_each_safe(client, tmp, &seat->clients, link) {
// will destroy other resources as well // will destroy other resources as well
wl_resource_destroy(client->wl_resource); wl_resource_destroy(client->wl_resource);
} }
wl_global_destroy(wlr_seat->wl_global); wl_global_destroy(seat->wl_global);
free(wlr_seat->pointer_state.default_grab); free(seat->pointer_state.default_grab);
free(wlr_seat->keyboard_state.default_grab); free(seat->keyboard_state.default_grab);
free(wlr_seat->touch_state.default_grab); free(seat->touch_state.default_grab);
free(wlr_seat->data_device); free(seat->data_device);
free(wlr_seat->name); free(seat->name);
free(wlr_seat); free(seat);
} }
static void handle_display_destroy(struct wl_listener *listener, void *data) { static void handle_display_destroy(struct wl_listener *listener, void *data) {