[PATCH weston v2 05/16] tablet: handle tablet cursors in the compositor

Maniraj Devadoss Maniraj.Devadoss at in.bosch.com
Thu Nov 23 10:17:17 UTC 2017


From: Lyude Paul <thatslyude at gmail.com>

The tablet is given a separate cursor. Most tablet interaction is an absolute
interaction and shouldn't need a cursor at all, but usually the cursor is used
to indicate the type of virtual tool currently assigned.

Co-authored-by: Peter Hutterer <peter.hutterer at who-t.net>
Signed-off-by: Lyude Paul <thatslyude at gmail.com>
Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
Signed-off-by: Bastian Farkas <bfarkas at de.adit-jv.com>

unmap the surface only if the sprite is not NULL

Signed-off-by: Maniraj Devadoss <Maniraj.Devadoss at in.bosch.com>
---
 libweston/compositor.c |   6 ++
 libweston/compositor.h |  13 ++++
 libweston/input.c      | 157 ++++++++++++++++++++++++++++++++++++++++++++++++-
 3 files changed, 175 insertions(+), 1 deletion(-)

diff --git a/libweston/compositor.c b/libweston/compositor.c
index 71eaf02..1c31b20 100644
--- a/libweston/compositor.c
+++ b/libweston/compositor.c
@@ -1778,6 +1778,7 @@ weston_view_unmap(struct weston_view *view)
 		struct weston_pointer *pointer = weston_seat_get_pointer(seat);
 		struct weston_keyboard *keyboard =
 			weston_seat_get_keyboard(seat);
+		struct weston_tablet_tool *tool;
 
 		if (keyboard && keyboard->focus == view->surface)
 			weston_keyboard_set_focus(keyboard, NULL);
@@ -1785,6 +1786,11 @@ weston_view_unmap(struct weston_view *view)
 			weston_pointer_clear_focus(pointer);
 		if (touch && touch->focus == view)
 			weston_touch_set_focus(touch, NULL);
+
+		wl_list_for_each(tool, &seat->tablet_tool_list, link) {
+			if (tool->focus == view)
+				weston_tablet_tool_set_focus(tool, NULL, 0);
+		}
 	}
 }
 
diff --git a/libweston/compositor.h b/libweston/compositor.h
index 49a0b56..95c2a2d 100644
--- a/libweston/compositor.h
+++ b/libweston/compositor.h
@@ -464,6 +464,12 @@ struct weston_tablet_tool {
 
 	int button_count;
 	bool tip_is_down;
+
+	int32_t hotspot_x, hotspot_y;
+	struct weston_view *sprite;
+	struct wl_listener sprite_destroy_listener;
+
+	wl_fixed_t x, y;
 };
 
 struct weston_tablet {
@@ -596,6 +602,13 @@ weston_tablet_tool_start_grab(struct weston_tablet_tool *tool,
 void
 weston_tablet_tool_end_grab(struct weston_tablet_tool *tool);
 
+void
+weston_tablet_tool_clamp(struct weston_tablet_tool *tool,
+			 wl_fixed_t *fx, wl_fixed_t *fy);
+void
+weston_tablet_tool_cursor_move(struct weston_tablet_tool *tool,
+			       wl_fixed_t x, wl_fixed_t y);
+
 bool
 weston_touch_has_focus_resource(struct weston_touch *touch);
 void
diff --git a/libweston/input.c b/libweston/input.c
index 1d863df..bfd6f55 100644
--- a/libweston/input.c
+++ b/libweston/input.c
@@ -1322,7 +1322,13 @@ static void
 default_grab_tablet_tool_proximity_out(struct weston_tablet_tool_grab *grab,
 				       uint32_t time)
 {
-	weston_tablet_tool_set_focus(grab->tool, NULL, time);
+	struct weston_tablet_tool *tool = grab->tool;
+
+	weston_tablet_tool_set_focus(tool, NULL, time);
+
+	/* Hide the cursor */
+	if (tool->sprite && weston_surface_is_mapped(tool->sprite->surface))
+		weston_surface_unmap(tool->sprite->surface);
 }
 
 static void
@@ -1337,6 +1343,8 @@ default_grab_tablet_tool_motion(struct weston_tablet_tool_grab *grab,
 	struct wl_resource *resource;
 	struct wl_list *resource_list = &tool->focus_resource_list;
 
+	weston_tablet_tool_cursor_move(tool, x, y);
+
 	current_view = weston_compositor_pick_view(tool->seat->compositor,
 						   x, y, &sx, &sy);
 	if (current_view != tool->focus)
@@ -1474,6 +1482,29 @@ static struct weston_tablet_tool_grab_interface default_tablet_tool_grab_interfa
 	default_grab_tablet_tool_cancel,
 };
 
+static void
+tablet_tool_unmap_sprite(struct weston_tablet_tool *tool)
+{
+	if (weston_surface_is_mapped(tool->sprite->surface))
+		weston_surface_unmap(tool->sprite->surface);
+
+	wl_list_remove(&tool->sprite_destroy_listener.link);
+	tool->sprite->surface->configure = NULL;
+	tool->sprite->surface->configure_private = NULL;
+	weston_view_destroy(tool->sprite);
+	tool->sprite = NULL;
+}
+
+static void
+tablet_tool_handle_sprite_destroy(struct wl_listener *listener, void *data)
+{
+	struct weston_tablet_tool *tool =
+		container_of(listener, struct weston_tablet_tool,
+			     sprite_destroy_listener);
+
+	tool->sprite = NULL;
+}
+
 WL_EXPORT struct weston_tablet_tool *
 weston_tablet_tool_create(void)
 {
@@ -1486,6 +1517,9 @@ weston_tablet_tool_create(void)
 	wl_list_init(&tool->resource_list);
 	wl_list_init(&tool->focus_resource_list);
 
+	wl_list_init(&tool->sprite_destroy_listener.link);
+	tool->sprite_destroy_listener.notify = tablet_tool_handle_sprite_destroy;
+
 	wl_list_init(&tool->focus_view_listener.link);
 	tool->focus_view_listener.notify = tablet_tool_focus_view_destroyed;
 
@@ -1504,12 +1538,57 @@ weston_tablet_tool_destroy(struct weston_tablet_tool *tool)
 {
 	struct wl_resource *resource, *tmp;
 
+	if (tool->sprite)
+		tablet_tool_unmap_sprite(tool);
+
 	wl_resource_for_each_safe(resource, tmp, &tool->resource_list)
 		zwp_tablet_tool_v1_send_removed(resource);
 
 	wl_list_remove(&tool->link);
 }
 
+WL_EXPORT void
+weston_tablet_tool_clamp(struct weston_tablet_tool *tool,
+			 wl_fixed_t *fx, wl_fixed_t *fy)
+{
+	struct weston_output *output = tool->current_tablet->output;
+	int x, y;
+
+	x = wl_fixed_to_int(*fx);
+	y = wl_fixed_to_int(*fy);
+
+	if (x < output->x)
+		*fx = wl_fixed_from_int(output->x);
+	else if (x >= output->x + output->width)
+		*fx = wl_fixed_from_int(output->x + output->width - 1);
+
+	if (y < output->y)
+		*fy = wl_fixed_from_int(output->y);
+	else if (y >= output->y + output->height)
+		*fy = wl_fixed_from_int(output->y + output->height - 1);
+}
+
+WL_EXPORT void
+weston_tablet_tool_cursor_move(struct weston_tablet_tool *tool,
+			       wl_fixed_t x,
+			       wl_fixed_t y)
+{
+	int32_t ix, iy;
+
+	weston_tablet_tool_clamp(tool, &x, &y);
+	tool->x = x;
+	tool->y = y;
+
+	ix = wl_fixed_to_int(x);
+	iy = wl_fixed_to_int(y);
+
+	if (tool->sprite) {
+		weston_view_set_position(tool->sprite,
+					 ix - tool->hotspot_x,
+					 iy - tool->hotspot_y);
+		weston_view_schedule_repaint(tool->sprite);
+	}
+}
 
 static void
 seat_send_updated_caps(struct weston_seat *seat)
@@ -2669,10 +2748,86 @@ notify_tablet_added(struct weston_tablet *tablet)
 }
 
 static void
+tablet_tool_cursor_surface_configure(struct weston_surface *es,
+				     int32_t dx, int32_t dy)
+{
+	struct weston_tablet_tool *tool = es->configure_private;
+	int x, y;
+
+	if (es->width == 0)
+		return;
+
+	assert(es == tool->sprite->surface);
+
+	tool->hotspot_x -= dx;
+	tool->hotspot_y -= dy;
+
+	x = wl_fixed_to_int(tool->x) - tool->hotspot_x;
+	y = wl_fixed_to_int(tool->y) - tool->hotspot_y;
+
+	weston_view_set_position(tool->sprite, x, y);
+
+	empty_region(&es->pending.input);
+	empty_region(&es->input);
+
+	if (!weston_surface_is_mapped(es)) {
+		weston_layer_entry_insert(
+			&es->compositor->cursor_layer.view_list,
+			&tool->sprite->layer_link);
+		weston_view_update_transform(tool->sprite);
+	}
+}
+
+static void
 tablet_tool_set_cursor(struct wl_client *client, struct wl_resource *resource,
 		       uint32_t serial, struct wl_resource *surface_resource,
 		       int32_t hotspot_x, int32_t hotspot_y)
 {
+	struct weston_tablet_tool *tool = wl_resource_get_user_data(resource);
+	struct weston_surface *surface = NULL;
+
+	if (surface_resource)
+		surface = wl_resource_get_user_data(surface_resource);
+
+	if (tool->focus == NULL)
+		return;
+
+	/* tablet->focus->surface->resource can be NULL. Surfaces like the
+	 * black_surface used in shell.c for fullscreen don't have
+	 * a resource, but can still have focus */
+	if (tool->focus->surface->resource == NULL)
+		return;
+
+	if (wl_resource_get_client(tool->focus->surface->resource) != client)
+		return;
+
+	if (tool->focus_serial - serial > UINT32_MAX / 2)
+		return;
+
+	if (surface && tool->sprite && surface != tool->sprite->surface &&
+	    surface->configure) {
+		wl_resource_post_error(surface->resource,
+				       WL_DISPLAY_ERROR_INVALID_OBJECT,
+				       "surface->configure already set");
+		return;
+	}
+
+	if (tool->sprite)
+		tablet_tool_unmap_sprite(tool);
+
+	if (!surface)
+		return;
+
+	wl_signal_add(&surface->destroy_signal,
+		      &tool->sprite_destroy_listener);
+	surface->configure = tablet_tool_cursor_surface_configure;
+	surface->configure_private = tool;
+	tool->sprite = weston_view_create(surface);
+	tool->hotspot_x = hotspot_x;
+	tool->hotspot_y = hotspot_y;
+
+	if (surface->buffer_ref.buffer)
+		tablet_tool_cursor_surface_configure(surface, 0, 0);
 }
 
 static void
-- 
2.7.4



More information about the wayland-devel mailing list