On Fri, 26 Jan 2018 18:47:59 +0200
Alexandros Frantzis <alexandros.frant...@collabora.com> wrote:

> The current test client code waits for all wl_seat globals to arrive
> before checking them and deciding which one is the test seat global to
> use for the input object. This method doesn't support dynamic addition
> of the test seat global (i.e., after client start-up), which will be
> needed in upcoming commits.
> 
> This commit changes the code to check for the test seat and set up the
> input object while handling the wl_seat information events.
> 
> Signed-off-by: Alexandros Frantzis <alexandros.frant...@collabora.com>
> ---
>  tests/weston-test-client-helper.c | 78 
> ++++++++++++++++++++++-----------------
>  tests/weston-test-client-helper.h |  1 +
>  2 files changed, 46 insertions(+), 33 deletions(-)

Hi Alexandros,

essentially patches 5-7 want to support dynamically creating and
removing wl_seats. The current test protocol is poorly suited for it as
it assumes the single test-seat in all requests. I would like to have a
protocol that better matches the structure of input devices and seats
and how weston core consumes input. However, that's a big task and it
would be outrageous to ask for that right here, so I think your
intention here is fine.

I presume the idea is that device_add("seat") and
device_release("seat") will create and destroy the test-seat,
respectively, regardless of the current capabilities.

> 
> diff --git a/tests/weston-test-client-helper.c 
> b/tests/weston-test-client-helper.c
> index 6e0a5246..854978d0 100644
> --- a/tests/weston-test-client-helper.c
> +++ b/tests/weston-test-client-helper.c

> @@ -862,9 +877,6 @@ create_client(void)
>        * events */
>       client_roundtrip(client);
>  
> -     /* find the right input for us */
> -     client_set_input(client);
> -

The original idea here was that the two roundtrips above guarantee that
we have processed all global advertisements and all wl_seat name and
capapbility events. Then we can ensure that Weston indeed provided
exactly one test-seat, and that all seats had a name provided.

I don't think that's in any way contradictory to allowing the test-seat
then to be removed and re-added, so I'm not quite sure why this patch
is needed or what it even does?


Thanks,
pq

Attachment: pgp_RDhDEiG1M.pgp
Description: OpenPGP digital signature

_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to