From e0b6af03cad9cb970072f39d602c85be6b4b014b Mon Sep 17 00:00:00 2001 From: Neil Roberts Date: Thu, 12 Jan 2012 15:48:02 +0000 Subject: [PATCH] server: In default grab, update focus resource after sending release The default grab implementation in wayland-server was updating the focus resource before sending the button event. This would cause the button release to be dropped from the implicit grab if the pointer is moved away from the focus surface. This patch just swaps the order around. --- src/wayland-server.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/wayland-server.c b/src/wayland-server.c index 514afb2..70adf28 100644 --- a/src/wayland-server.c +++ b/src/wayland-server.c @@ -445,17 +445,17 @@ default_grab_button(struct wl_grab *grab, struct wl_input_device *device = grab->input_device; struct wl_resource *resource; + resource = device->pointer_focus_resource; + if (resource) + wl_resource_post_event(resource, WL_INPUT_DEVICE_BUTTON, + time, button, state); + if (device->button_count == 0 && state == 0) wl_input_device_set_pointer_focus(device, device->current, time, device->x, device->y, device->current_x, device->current_y); - - resource = device->pointer_focus_resource; - if (resource) - wl_resource_post_event(resource, WL_INPUT_DEVICE_BUTTON, - time, button, state); } static const struct wl_grab_interface default_grab_interface = {