[PATCH weston 4/4] input: Emit events on all resources for a client

Rob Bradford robert.bradford at intel.com
Fri Sep 6 09:48:22 PDT 2013


From: Rob Bradford <rob at linux.intel.com>

The Wayland protocol permits a client to request the pointer, keyboard
and touch multiple times from the seat global. This is very useful in a
component like Clutter-GTK where we are combining two libraries that use
Wayland together.

This change migrates the weston input handling code to emit the
events for all the resources for the client by using the newly added
wl_resource_for_each macro to iterate over the resources that are
associated with the focussed surface's client.

We maintain a list of focussed resources on the pointer and keyboard
which is updated when the focus changes. However since we can have
resources created after the focus has already been set we must add the
resources to the right list and also update any state.
---
 src/bindings.c   |  17 ++-
 src/compositor.h |   6 +-
 src/input.c      | 344 +++++++++++++++++++++++++++++++++++--------------------
 src/shell.c      |  33 +++---
 4 files changed, 248 insertions(+), 152 deletions(-)

diff --git a/src/bindings.c b/src/bindings.c
index a871c26..55823e0 100644
--- a/src/bindings.c
+++ b/src/bindings.c
@@ -160,7 +160,6 @@ binding_key(struct weston_keyboard_grab *grab,
 	struct weston_keyboard *keyboard = grab->keyboard;
 	struct wl_display *display = keyboard->seat->compositor->wl_display;
 
-	resource = grab->keyboard->focus_resource;
 	if (key == b->key) {
 		if (state == WL_KEYBOARD_KEY_STATE_RELEASED) {
 			weston_keyboard_end_grab(grab->keyboard);
@@ -168,9 +167,11 @@ binding_key(struct weston_keyboard_grab *grab,
 				keyboard->grab = &keyboard->input_method_grab;
 			free(b);
 		}
-	} else if (resource) {
+	} else {
 		serial = wl_display_next_serial(display);
-		wl_keyboard_send_key(resource, serial, time, key, state);
+		wl_resource_for_each(resource, &keyboard->focus_resource_list) {
+			wl_keyboard_send_key(resource, serial, time, key, state);
+		}
 	}
 }
 
@@ -181,12 +182,10 @@ binding_modifiers(struct weston_keyboard_grab *grab, uint32_t serial,
 {
 	struct wl_resource *resource;
 
-	resource = grab->keyboard->focus_resource;
-	if (!resource)
-		return;
-
-	wl_keyboard_send_modifiers(resource, serial, mods_depressed,
-				   mods_latched, mods_locked, group);
+	wl_resource_for_each(resource, &grab->keyboard->focus_resource_list) {
+		wl_keyboard_send_modifiers(resource, serial, mods_depressed,
+					   mods_latched, mods_locked, group);
+	}
 }
 
 static const struct weston_keyboard_grab_interface binding_grab = {
diff --git a/src/compositor.h b/src/compositor.h
index 6db3c61..23201cf 100644
--- a/src/compositor.h
+++ b/src/compositor.h
@@ -300,8 +300,8 @@ struct weston_pointer {
 	struct weston_seat *seat;
 
 	struct wl_list resource_list;
+	struct wl_list focus_resource_list;
 	struct weston_surface *focus;
-	struct wl_resource *focus_resource;
 	struct wl_listener focus_listener;
 	uint32_t focus_serial;
 	struct wl_signal focus_signal;
@@ -326,8 +326,8 @@ struct weston_touch {
 	struct weston_seat *seat;
 
 	struct wl_list resource_list;
+	struct wl_list focus_resource_list;
 	struct weston_surface *focus;
-	struct wl_resource *focus_resource;
 	struct wl_listener focus_listener;
 	uint32_t focus_serial;
 	struct wl_signal focus_signal;
@@ -415,8 +415,8 @@ struct weston_keyboard {
 	struct weston_seat *seat;
 
 	struct wl_list resource_list;
+	struct wl_list focus_resource_list;
 	struct weston_surface *focus;
-	struct wl_resource *focus_resource;
 	struct wl_listener focus_listener;
 	uint32_t focus_serial;
 	struct wl_signal focus_signal;
diff --git a/src/input.c b/src/input.c
index 7344c4d..d63ca77 100644
--- a/src/input.c
+++ b/src/input.c
@@ -71,12 +71,33 @@ weston_compositor_idle_release(struct weston_compositor *compositor)
 }
 
 static void
+move_resources(struct wl_list *destination, struct wl_list *source)
+{
+	struct wl_resource *resource, *tmp;
+	wl_resource_for_each_safe(resource, tmp, source) {
+		wl_list_remove(wl_resource_get_link(resource));
+		wl_list_insert(destination, wl_resource_get_link(resource));
+	}
+}
+
+static void
+move_resources_for_client(struct wl_list *destination, struct wl_list *source, struct wl_client *client)
+{
+	struct wl_resource *resource, *tmp;
+	wl_resource_for_each_safe(resource, tmp, source) {
+		if (wl_resource_get_client(resource) == client) {
+			wl_list_remove(wl_resource_get_link(resource));
+			wl_list_insert(destination, wl_resource_get_link(resource));
+		}
+	}
+}
+
+static void
 lose_pointer_focus(struct wl_listener *listener, void *data)
 {
 	struct weston_pointer *pointer =
 		container_of(listener, struct weston_pointer, focus_listener);
-
-	pointer->focus_resource = NULL;
+	move_resources(&pointer->resource_list, &pointer->focus_resource_list);
 }
 
 static void
@@ -84,8 +105,7 @@ lose_keyboard_focus(struct wl_listener *listener, void *data)
 {
 	struct weston_keyboard *keyboard =
 		container_of(listener, struct weston_keyboard, focus_listener);
-
-	keyboard->focus_resource = NULL;
+	move_resources(&keyboard->resource_list, &keyboard->focus_resource_list);
 }
 
 static void
@@ -93,8 +113,7 @@ lose_touch_focus(struct wl_listener *listener, void *data)
 {
 	struct weston_touch *touch =
 		container_of(listener, struct weston_touch, focus_listener);
-
-	touch->focus_resource = NULL;
+	move_resources(&touch->resource_list, &touch->focus_resource_list);
 }
 
 static void
@@ -120,12 +139,15 @@ default_grab_motion(struct weston_pointer_grab *grab, uint32_t time)
 {
 	struct weston_pointer *pointer = grab->pointer;
 	wl_fixed_t sx, sy;
+	struct wl_list *resource_list;
+	struct wl_resource *resource;
 
-	if (pointer->focus_resource) {
+	resource_list = &pointer->focus_resource_list;
+	wl_resource_for_each(resource, resource_list) {
 		weston_surface_from_global_fixed(pointer->focus,
 						 pointer->x, pointer->y,
 						 &sx, &sy);
-		wl_pointer_send_motion(pointer->focus_resource, time, sx, sy);
+		wl_pointer_send_motion(resource, time, sx, sy);
 	}
 }
 
@@ -141,11 +163,13 @@ default_grab_button(struct weston_pointer_grab *grab,
 	enum wl_pointer_button_state state = state_w;
 	struct wl_display *display = compositor->wl_display;
 	wl_fixed_t sx, sy;
+	struct wl_list *resource_list;
 
-	resource = pointer->focus_resource;
-	if (resource) {
+	resource_list = &pointer->focus_resource_list;
+	if (!wl_list_empty(resource_list)) {
 		serial = wl_display_next_serial(display);
-		wl_pointer_send_button(resource, serial, time, button, state_w);
+		wl_resource_for_each(resource, resource_list)
+			wl_pointer_send_button(resource, serial, time, button, state_w);
 	}
 
 	if (pointer->button_count == 0 &&
@@ -173,12 +197,17 @@ default_grab_touch_down(struct weston_touch_grab *grab, uint32_t time,
 	struct weston_touch *touch = grab->touch;
 	struct wl_display *display = touch->seat->compositor->wl_display;
 	uint32_t serial;
+	struct wl_resource *resource;
+	struct wl_list *resource_list;
 
-	if (touch->focus_resource && touch->focus) {
+	resource_list = &touch->focus_resource_list;
+
+	if (!wl_list_empty(resource_list) && touch->focus) {
 		serial = wl_display_next_serial(display);
-		wl_touch_send_down(touch->focus_resource, serial, time,
-				   touch->focus->resource,
-				   touch_id, sx, sy);
+		wl_resource_for_each(resource, resource_list)
+				wl_touch_send_down(resource, serial, time,
+						   touch->focus->resource,
+						   touch_id, sx, sy);
 	}
 }
 
@@ -189,10 +218,15 @@ default_grab_touch_up(struct weston_touch_grab *grab,
 	struct weston_touch *touch = grab->touch;
 	struct wl_display *display = touch->seat->compositor->wl_display;
 	uint32_t serial;
+	struct wl_resource *resource;
+	struct wl_list *resource_list;
 
-	if (touch->focus_resource) {
+	resource_list = &touch->focus_resource_list;
+
+	if (!wl_list_empty(resource_list)) {
 		serial = wl_display_next_serial(display);
-		wl_touch_send_up(touch->focus_resource, serial, time, touch_id);
+		wl_resource_for_each(resource, resource_list)
+			wl_touch_send_up(resource, serial, time, touch_id);
 	}
 }
 
@@ -201,9 +235,13 @@ default_grab_touch_motion(struct weston_touch_grab *grab, uint32_t time,
 			  int touch_id, wl_fixed_t sx, wl_fixed_t sy)
 {
 	struct weston_touch *touch = grab->touch;
+	struct wl_resource *resource;
+	struct wl_list *resource_list;
+
+	resource_list = &touch->focus_resource_list;
 
-	if (touch->focus_resource) {
-		wl_touch_send_motion(touch->focus_resource, time,
+	wl_resource_for_each(resource, resource_list) {
+		wl_touch_send_motion(resource, time,
 				     touch_id, sx, sy);
 	}
 }
@@ -222,52 +260,42 @@ default_grab_key(struct weston_keyboard_grab *grab,
 	struct wl_resource *resource;
 	struct wl_display *display = keyboard->seat->compositor->wl_display;
 	uint32_t serial;
+	struct wl_list *resource_list;
 
-	resource = keyboard->focus_resource;
-	if (resource) {
+	resource_list = &keyboard->focus_resource_list;
+	wl_resource_for_each(resource, resource_list) {
 		serial = wl_display_next_serial(display);
 		wl_keyboard_send_key(resource, serial, time, key, state);
 	}
 }
 
-static struct wl_resource *
-find_resource_for_surface(struct wl_list *list, struct weston_surface *surface)
-{
-	if (!surface)
-		return NULL;
-
-	if (!surface->resource)
-		return NULL;
-	
-	return wl_resource_find_for_client(list, wl_resource_get_client(surface->resource));
-}
-
 static void
 default_grab_modifiers(struct weston_keyboard_grab *grab, uint32_t serial,
 		       uint32_t mods_depressed, uint32_t mods_latched,
 		       uint32_t mods_locked, uint32_t group)
 {
 	struct weston_keyboard *keyboard = grab->keyboard;
-	struct weston_pointer *pointer = keyboard->seat->pointer;
-	struct wl_resource *resource, *pr;
-
-	resource = keyboard->focus_resource;
-	if (!resource)
-		return;
+	struct weston_pointer *pointer = grab->keyboard->seat->pointer;
+	struct wl_resource *resource;
+	struct wl_list *resource_list;
 
-	wl_keyboard_send_modifiers(resource, serial, mods_depressed,
-				   mods_latched, mods_locked, group);
+	resource_list = &keyboard->focus_resource_list;
 
+	wl_resource_for_each(resource, resource_list) {
+		wl_keyboard_send_modifiers(resource, serial, mods_depressed,
+					   mods_latched, mods_locked, group);
+	}
 	if (pointer && pointer->focus && pointer->focus != keyboard->focus) {
-		pr = find_resource_for_surface(&keyboard->resource_list,
-					       pointer->focus);
-		if (pr) {
-			wl_keyboard_send_modifiers(pr,
-						   serial,
-						   keyboard->modifiers.mods_depressed,
-						   keyboard->modifiers.mods_latched,
-						   keyboard->modifiers.mods_locked,
-						   keyboard->modifiers.group);
+		wl_resource_for_each(resource, &keyboard->resource_list) {
+			if (wl_resource_get_client(resource) ==
+			    wl_resource_get_client(pointer->focus->resource)) {
+				wl_keyboard_send_modifiers(resource,
+							   serial,
+							   keyboard->modifiers.mods_depressed,
+							   keyboard->modifiers.mods_latched,
+							   keyboard->modifiers.mods_locked,
+							   keyboard->modifiers.group);
+			}
 		}
 	}
 }
@@ -310,6 +338,7 @@ weston_pointer_create(void)
 		return NULL;
 
 	wl_list_init(&pointer->resource_list);
+	wl_list_init(&pointer->focus_resource_list);
 	pointer->focus_listener.notify = lose_pointer_focus;
 	pointer->default_grab.interface = &default_pointer_grab_interface;
 	pointer->default_grab.pointer = pointer;
@@ -332,7 +361,7 @@ weston_pointer_destroy(struct weston_pointer *pointer)
 		pointer_unmap_sprite(pointer);
 
 	/* XXX: What about pointer->resource_list? */
-	if (pointer->focus_resource)
+	if (!wl_list_empty(&pointer->focus_resource_list))
 		wl_list_remove(&pointer->focus_listener.link);
 	free(pointer);
 }
@@ -347,6 +376,7 @@ weston_keyboard_create(void)
 	    return NULL;
 
 	wl_list_init(&keyboard->resource_list);
+	wl_list_init(&keyboard->focus_resource_list);
 	wl_array_init(&keyboard->keys);
 	keyboard->focus_listener.notify = lose_keyboard_focus;
 	keyboard->default_grab.interface = &default_keyboard_grab_interface;
@@ -361,7 +391,7 @@ WL_EXPORT void
 weston_keyboard_destroy(struct weston_keyboard *keyboard)
 {
 	/* XXX: What about keyboard->resource_list? */
-	if (keyboard->focus_resource)
+	if (!wl_list_empty(&keyboard->focus_resource_list))
 		wl_list_remove(&keyboard->focus_listener.link);
 	wl_array_release(&keyboard->keys);
 	free(keyboard);
@@ -377,6 +407,7 @@ weston_touch_create(void)
 		return NULL;
 
 	wl_list_init(&touch->resource_list);
+	wl_list_init(&touch->focus_resource_list);
 	touch->focus_listener.notify = lose_touch_focus;
 	touch->default_grab.interface = &default_touch_grab_interface;
 	touch->default_grab.touch = touch;
@@ -390,7 +421,7 @@ WL_EXPORT void
 weston_touch_destroy(struct weston_touch *touch)
 {
 	/* XXX: What about touch->resource_list? */
-	if (touch->focus_resource)
+	if (!wl_list_empty(&touch->focus_resource_list))
 		wl_list_remove(&touch->focus_listener.link);
 	free(touch);
 }
@@ -422,42 +453,51 @@ weston_pointer_set_focus(struct weston_pointer *pointer,
 	struct wl_resource *resource, *kr;
 	struct wl_display *display = pointer->seat->compositor->wl_display;
 	uint32_t serial;
+	struct wl_list *focus_resource_list;
 
-	resource = pointer->focus_resource;
-	if (resource && pointer->focus != surface) {
+	focus_resource_list = &pointer->focus_resource_list;
+
+	if (!wl_list_empty(focus_resource_list) && pointer->focus != surface) {
 		serial = wl_display_next_serial(display);
-		wl_pointer_send_leave(resource, serial,
-				      pointer->focus->resource);
+		wl_resource_for_each(resource, focus_resource_list) {
+			wl_pointer_send_leave(resource, serial,
+					      pointer->focus->resource);
+		}
+
+		move_resources(&pointer->resource_list, focus_resource_list);
 		wl_list_remove(&pointer->focus_listener.link);
 	}
 
-	resource = find_resource_for_surface(&pointer->resource_list,
-					     surface);
-	if (resource &&
-	    (pointer->focus != surface ||
-	     pointer->focus_resource != resource)) {
+	if (pointer->focus != surface && wl_list_empty (focus_resource_list)) {
 		serial = wl_display_next_serial(display);
-		if (kbd) {
-			kr = find_resource_for_surface(&kbd->resource_list,
-						       surface);
-			if (kr) {
-				wl_keyboard_send_modifiers(kr,
-							   serial,
-							   kbd->modifiers.mods_depressed,
-							   kbd->modifiers.mods_latched,
-							   kbd->modifiers.mods_locked,
-							   kbd->modifiers.group);
+		if (kbd && surface && kbd->focus != pointer->focus) {
+			wl_resource_for_each(kr, &kbd->resource_list) {
+				if (wl_resource_get_client(kr) == wl_resource_get_client(surface->resource)) {
+					wl_keyboard_send_modifiers(kr,
+								   serial,
+								   kbd->modifiers.mods_depressed,
+								   kbd->modifiers.mods_latched,
+								   kbd->modifiers.mods_locked,
+								   kbd->modifiers.group);
+				}
 			}
 		}
-		wl_pointer_send_enter(resource, serial, surface->resource,
-				      sx, sy);
-		wl_client_add_destroy_listener(wl_resource_get_client(surface->resource),
-					       &pointer->focus_listener);
-		pointer->focus_serial = serial;
+		if (surface) {
+			move_resources_for_client(focus_resource_list, &pointer->resource_list,
+						  wl_resource_get_client(surface->resource));
+
+			wl_resource_for_each(resource, focus_resource_list) {
+				wl_pointer_send_enter(resource, serial, surface->resource,
+						      sx, sy);
+			}
+
+			wl_client_add_destroy_listener(wl_resource_get_client(surface->resource),
+						       &pointer->focus_listener);
+			pointer->focus_serial = serial;
+		}
+		pointer->focus = surface;
 	}
 
-	pointer->focus_resource = resource;
-	pointer->focus = surface;
 	wl_signal_emit(&pointer->focus_signal, pointer);
 }
 
@@ -468,35 +508,42 @@ weston_keyboard_set_focus(struct weston_keyboard *keyboard,
 	struct wl_resource *resource;
 	struct wl_display *display = keyboard->seat->compositor->wl_display;
 	uint32_t serial;
+	struct wl_list *focus_resource_list;
+
+	focus_resource_list = &keyboard->focus_resource_list;
 
-	if (keyboard->focus_resource && keyboard->focus != surface) {
-		resource = keyboard->focus_resource;
+	if (!wl_list_empty(focus_resource_list) && keyboard->focus != surface) {
 		serial = wl_display_next_serial(display);
-		wl_keyboard_send_leave(resource, serial,
-				       keyboard->focus->resource);
+		wl_resource_for_each(resource, focus_resource_list) {
+			wl_keyboard_send_leave(resource, serial,
+					keyboard->focus->resource);
+		}
+		move_resources(&keyboard->resource_list, focus_resource_list);
 		wl_list_remove(&keyboard->focus_listener.link);
 	}
 
-	resource = find_resource_for_surface(&keyboard->resource_list,
-					     surface);
-	if (resource &&
-	    (keyboard->focus != surface ||
-	     keyboard->focus_resource != resource)) {
+	if (keyboard->focus != surface && wl_list_empty(focus_resource_list)) {
 		serial = wl_display_next_serial(display);
-		wl_keyboard_send_modifiers(resource, serial,
-					   keyboard->modifiers.mods_depressed,
-					   keyboard->modifiers.mods_latched,
-					   keyboard->modifiers.mods_locked,
-					   keyboard->modifiers.group);
-		wl_keyboard_send_enter(resource, serial, surface->resource,
-				       &keyboard->keys);
-		wl_client_add_destroy_listener(wl_resource_get_client(surface->resource),
-					       &keyboard->focus_listener);
-		keyboard->focus_serial = serial;
-	}
 
-	keyboard->focus_resource = resource;
-	keyboard->focus = surface;
+		if (surface) {
+			move_resources_for_client(focus_resource_list, &keyboard->resource_list,
+						  wl_resource_get_client(surface->resource));
+			wl_resource_for_each(resource, focus_resource_list) {
+				wl_keyboard_send_modifiers(resource, serial,
+							   keyboard->modifiers.mods_depressed,
+							   keyboard->modifiers.mods_latched,
+							   keyboard->modifiers.mods_locked,
+							   keyboard->modifiers.group);
+				wl_keyboard_send_enter(resource, serial, surface->resource,
+						       &keyboard->keys);
+			}
+			wl_client_add_destroy_listener(wl_resource_get_client(surface->resource),
+						       &keyboard->focus_listener);
+			keyboard->focus_serial = serial;
+		}
+		keyboard->focus = surface;
+}
+
 	wl_signal_emit(&keyboard->focus_signal, keyboard);
 }
 
@@ -705,6 +752,8 @@ notify_axis(struct weston_seat *seat, uint32_t time, uint32_t axis,
 	struct weston_surface *focus =
 		(struct weston_surface *) pointer->focus;
 	uint32_t serial = wl_display_next_serial(compositor->wl_display);
+	struct wl_resource *resource;
+	struct wl_list *resource_list;
 
 	if (compositor->ping_handler && focus)
 		compositor->ping_handler(focus, serial);
@@ -718,8 +767,9 @@ notify_axis(struct weston_seat *seat, uint32_t time, uint32_t axis,
 						   time, axis, value))
 		return;
 
-	if (pointer->focus_resource)
-		wl_pointer_send_axis(pointer->focus_resource, time, axis,
+	resource_list = &pointer->focus_resource_list;
+	wl_resource_for_each(resource, resource_list)
+		wl_pointer_send_axis(resource, time, axis,
 				     value);
 }
 
@@ -976,29 +1026,27 @@ notify_keyboard_focus_out(struct weston_seat *seat)
 WL_EXPORT void
 weston_touch_set_focus(struct weston_seat *seat, struct weston_surface *surface)
 {
-	struct wl_resource *resource;
+	struct wl_list *focus_resource_list;
+
+	focus_resource_list = &seat->touch->focus_resource_list;
 
 	if (seat->touch->focus == surface)
 		return;
 
-	if (seat->touch->focus_resource)
+	if (!wl_list_empty(focus_resource_list) && seat->touch->focus != surface) {
+		move_resources(&seat->touch->resource_list, focus_resource_list);
 		wl_list_remove(&seat->touch->focus_listener.link);
-	seat->touch->focus = NULL;
-	seat->touch->focus_resource = NULL;
+	}
 
-	if (surface) {
-		resource =
-			find_resource_for_surface(&seat->touch->resource_list,
-						  surface);
-		if (!resource) {
-			weston_log("couldn't find resource\n");
-			return;
-		}
+	if (seat->touch->focus != surface) {
+		if (surface) {
+			move_resources_for_client(focus_resource_list, &seat->touch->resource_list,
+						  wl_resource_get_client(surface->resource));
+			wl_client_add_destroy_listener(wl_resource_get_client(surface->resource),
+						       &seat->touch->focus_listener);
 
-		seat->touch->focus = surface;
-		seat->touch->focus_resource = resource;
-		wl_client_add_destroy_listener(wl_resource_get_client(surface->resource),
-					       &seat->touch->focus_listener);
+		}
+	seat->touch->focus = surface;
 	}
 }
 
@@ -1187,6 +1235,9 @@ seat_get_pointer(struct wl_client *client, struct wl_resource *resource,
 		return;
 	}
 
+	/* May be moved to focussed list later by either
+	 * weston_pointer_set_focus or directly if this client is already
+	 * focussed */
 	wl_list_insert(&seat->pointer->resource_list, wl_resource_get_link(cr));
 	wl_resource_set_implementation(cr, &pointer_interface, seat->pointer,
 				       unbind_resource);
@@ -1202,10 +1253,22 @@ seat_get_pointer(struct wl_client *client, struct wl_resource *resource,
 						 seat->pointer->y,
 						 &sx,
 						 &sy);
-		weston_pointer_set_focus(seat->pointer,
-					 seat->pointer->focus,
-					 sx,
-					 sy);
+
+		/* First input resource created for this client */
+		if (wl_list_empty(&seat->pointer->focus_resource_list)) {
+			weston_pointer_set_focus(seat->pointer,
+						 seat->pointer->focus,
+						 sx,
+						 sy);
+		} else {
+			uint32_t serial;
+			serial = wl_display_next_serial(seat->compositor->wl_display);
+			wl_list_remove(wl_resource_get_link(cr));
+			wl_list_insert(&seat->pointer->focus_resource_list, wl_resource_get_link(cr));
+			wl_pointer_send_enter(cr, serial, surface->resource,
+					      sx, sy);
+			seat->pointer->focus_serial = serial;
+		}
 	}
 }
 
@@ -1236,6 +1299,9 @@ seat_get_keyboard(struct wl_client *client, struct wl_resource *resource,
 		return;
 	}
 
+	/* May be moved to focussed list later by either
+	 * weston_keyboard_set_focus or directly if this client is already
+	 * focussed */
 	wl_list_insert(&seat->keyboard->resource_list, wl_resource_get_link(cr));
 	wl_resource_set_implementation(cr, &keyboard_interface,
 				       seat, unbind_resource);
@@ -1254,8 +1320,29 @@ seat_get_keyboard(struct wl_client *client, struct wl_resource *resource,
 
 	if (seat->keyboard->focus &&
 	    wl_resource_get_client(seat->keyboard->focus->resource) == client) {
-		weston_keyboard_set_focus(seat->keyboard,
-					  seat->keyboard->focus);
+		struct weston_surface *surface;
+
+		surface = (struct weston_surface *) seat->keyboard->focus;
+
+		/* First input resource created for this client */
+		if (wl_list_empty(&seat->keyboard->focus_resource_list)) {
+			weston_keyboard_set_focus(seat->keyboard,
+						  seat->keyboard->focus);
+		} else {
+			uint32_t serial;
+
+			serial = wl_display_next_serial(seat->compositor->wl_display);
+			wl_list_remove(wl_resource_get_link(cr));
+			wl_list_insert(&seat->keyboard->focus_resource_list, wl_resource_get_link(cr));
+			wl_keyboard_send_modifiers(cr, serial,
+						   seat->keyboard->modifiers.mods_depressed,
+						   seat->keyboard->modifiers.mods_latched,
+						   seat->keyboard->modifiers.mods_locked,
+						   seat->keyboard->modifiers.group);
+			wl_keyboard_send_enter(cr, serial, surface->resource,
+					       &seat->keyboard->keys);
+			seat->keyboard->focus_serial = serial;
+		}
 		wl_data_device_set_keyboard_focus(seat);
 	}
 }
@@ -1287,7 +1374,12 @@ seat_get_touch(struct wl_client *client, struct wl_resource *resource,
 		return;
 	}
 
-	wl_list_insert(&seat->touch->resource_list, wl_resource_get_link(cr));
+	if (seat->touch->focus &&
+	    wl_resource_get_client(seat->touch->focus->resource) == client) {
+		wl_list_insert(&seat->touch->resource_list, wl_resource_get_link(cr));
+	} else {
+		wl_list_insert(&seat->touch->focus_resource_list, wl_resource_get_link(cr));
+	}
 	wl_resource_set_implementation(cr, &touch_interface,
 				       seat, unbind_resource);
 }
diff --git a/src/shell.c b/src/shell.c
index cd94aa5..59e2821 100644
--- a/src/shell.c
+++ b/src/shell.c
@@ -2155,13 +2155,14 @@ static void
 popup_grab_motion(struct weston_pointer_grab *grab, uint32_t time)
 {
 	struct weston_pointer *pointer = grab->pointer;
+	struct wl_resource *resource;
 	wl_fixed_t sx, sy;
 
-	if (pointer->focus_resource) {
+	wl_resource_for_each(resource, &pointer->focus_resource_list) {
 		weston_surface_from_global_fixed(pointer->focus,
 						 pointer->x, pointer->y,
 						 &sx, &sy);
-		wl_pointer_send_motion(pointer->focus_resource, time, sx, sy);
+		wl_pointer_send_motion(resource, time, sx, sy);
 	}
 }
 
@@ -2175,11 +2176,14 @@ popup_grab_button(struct weston_pointer_grab *grab,
 	struct wl_display *display = shseat->seat->compositor->wl_display;
 	enum wl_pointer_button_state state = state_w;
 	uint32_t serial;
+	struct wl_list *resource_list;
 
-	resource = grab->pointer->focus_resource;
-	if (resource) {
+	resource_list = &grab->pointer->focus_resource_list;
+	if (!wl_list_empty(resource_list)) {
 		serial = wl_display_get_serial(display);
-		wl_pointer_send_button(resource, serial, time, button, state);
+		wl_resource_for_each (resource, resource_list) {
+			wl_pointer_send_button(resource, serial, time, button, state);
+		}
 	} else if (state == WL_POINTER_BUTTON_STATE_RELEASED &&
 		   (shseat->popup_grab.initial_up ||
 		    time - shseat->seat->pointer->grab_time > 500)) {
@@ -4247,6 +4251,7 @@ debug_binding_key(struct weston_keyboard_grab *grab, uint32_t time,
 	int send = 0, terminate = 0;
 	int check_binding = 1;
 	int i;
+	struct wl_list *resource_list;
 
 	if (state == WL_KEYBOARD_KEY_STATE_RELEASED) {
 		/* Do not run bindings on key releases */
@@ -4293,10 +4298,9 @@ debug_binding_key(struct weston_keyboard_grab *grab, uint32_t time,
 	}
 
 	if (send) {
-		resource = grab->keyboard->focus_resource;
-
-		if (resource) {
-			serial = wl_display_next_serial(display);
+		serial = wl_display_next_serial(display);
+		resource_list = &grab->keyboard->focus_resource_list;
+		wl_resource_for_each(resource, resource_list) {
 			wl_keyboard_send_key(resource, serial, time, key, state);
 		}
 	}
@@ -4315,13 +4319,14 @@ debug_binding_modifiers(struct weston_keyboard_grab *grab, uint32_t serial,
 			uint32_t mods_locked, uint32_t group)
 {
 	struct wl_resource *resource;
+	struct wl_list *resource_list;
 
-	resource = grab->keyboard->focus_resource;
-	if (!resource)
-		return;
+	resource_list = &grab->keyboard->focus_resource_list;
 
-	wl_keyboard_send_modifiers(resource, serial, mods_depressed,
-				   mods_latched, mods_locked, group);
+	wl_resource_for_each(resource, resource_list) {
+		wl_keyboard_send_modifiers(resource, serial, mods_depressed,
+								   mods_latched, mods_locked, group);
+	}
 }
 
 struct weston_keyboard_grab_interface debug_binding_keyboard_grab = {
-- 
1.8.3.1



More information about the wayland-devel mailing list