clients: Set the hotspot with attach if we already have a valid cursor

We want atomic hotspot updates - this can't happen with
wl_pointer_set_cursor. So if we have a surface that already has a cursor
role, just update the hotspot when attaching new content.

Signed-off-by: Derek Foreman <derek.foreman@collabora.com>
(cherry picked from commit 646cc1b389)
dev
Derek Foreman 2 years ago committed by Marius Vlad
parent c32e3c6bc0
commit 8f8ff5f294
  1. 29
      clients/window.c

@ -352,6 +352,8 @@ struct input {
struct toytimer cursor_timer; struct toytimer cursor_timer;
bool cursor_timer_running; bool cursor_timer_running;
struct wl_surface *pointer_surface; struct wl_surface *pointer_surface;
bool pointer_surface_has_role;
int hotspot_x, hotspot_y;
uint32_t modifiers; uint32_t modifiers;
uint32_t pointer_enter_serial; uint32_t pointer_enter_serial;
uint32_t cursor_serial; uint32_t cursor_serial;
@ -2776,6 +2778,7 @@ pointer_handle_enter(void *data, struct wl_pointer *pointer,
input->display->serial = serial; input->display->serial = serial;
input->pointer_enter_serial = serial; input->pointer_enter_serial = serial;
input->pointer_focus = window; input->pointer_focus = window;
input->pointer_surface_has_role = false;
input->sx = sx; input->sx = sx;
input->sy = sy; input->sy = sy;
@ -3788,6 +3791,7 @@ input_set_pointer_image_index(struct input *input, int index)
struct wl_buffer *buffer; struct wl_buffer *buffer;
struct wl_cursor *cursor; struct wl_cursor *cursor;
struct wl_cursor_image *image; struct wl_cursor_image *image;
int dx = 0, dy = 0;
if (!input->pointer) if (!input->pointer)
return; return;
@ -3806,13 +3810,24 @@ input_set_pointer_image_index(struct input *input, int index)
if (!buffer) if (!buffer)
return; return;
wl_surface_attach(input->pointer_surface, buffer, 0, 0); if (input->pointer_surface_has_role) {
dx = input->hotspot_x - image->hotspot_x;
dy = input->hotspot_y - image->hotspot_y;
}
wl_surface_attach(input->pointer_surface, buffer, dx, dy);
wl_surface_damage(input->pointer_surface, 0, 0, wl_surface_damage(input->pointer_surface, 0, 0,
image->width, image->height); image->width, image->height);
wl_surface_commit(input->pointer_surface); wl_surface_commit(input->pointer_surface);
wl_pointer_set_cursor(input->pointer, input->pointer_enter_serial,
input->pointer_surface, if (!input->pointer_surface_has_role) {
image->hotspot_x, image->hotspot_y); wl_pointer_set_cursor(input->pointer,
input->pointer_enter_serial,
input->pointer_surface,
image->hotspot_x, image->hotspot_y);
input->pointer_surface_has_role = true;
}
input->hotspot_x = image->hotspot_x;
input->hotspot_y = image->hotspot_y;
} }
static const struct wl_callback_listener pointer_surface_listener; static const struct wl_callback_listener pointer_surface_listener;
@ -3824,11 +3839,14 @@ input_set_pointer_special(struct input *input)
wl_pointer_set_cursor(input->pointer, wl_pointer_set_cursor(input->pointer,
input->pointer_enter_serial, input->pointer_enter_serial,
NULL, 0, 0); NULL, 0, 0);
input->pointer_surface_has_role = false;
return true; return true;
} }
if (input->current_cursor == CURSOR_UNSET) if (input->current_cursor == CURSOR_UNSET) {
input->pointer_surface_has_role = false;
return true; return true;
}
return false; return false;
} }
@ -5923,6 +5941,7 @@ display_add_input(struct display *d, uint32_t id, int display_seat_version)
} }
input->pointer_surface = wl_compositor_create_surface(d->compositor); input->pointer_surface = wl_compositor_create_surface(d->compositor);
input->pointer_surface_has_role = false;
toytimer_init(&input->cursor_timer, CLOCK_MONOTONIC, d, toytimer_init(&input->cursor_timer, CLOCK_MONOTONIC, d,
cursor_timer_func); cursor_timer_func);

Loading…
Cancel
Save