[PATCH weston] compositor: Always empty the input region of the pointer surface

Kristian Høgsberg hoegsberg at gmail.com
Mon Jul 9 14:32:56 PDT 2012


On Wed, Jul 04, 2012 at 03:48:29PM +0300, Ander Conselvan de Oliveira wrote:
> Otherwise a surface.attach request might cause the input region to be
> reset to the default value (the entire surface) causing it to receive
> focus.

Thanks, applied.
Kristian

> Tiago ran into this problem with xwayland.
> ---
>  src/compositor.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/src/compositor.c b/src/compositor.c
> index 677b5c8..678e0bd 100644
> --- a/src/compositor.c
> +++ b/src/compositor.c
> @@ -2145,11 +2145,12 @@ pointer_cursor_surface_configure(struct weston_surface *es,
>  	weston_surface_configure(seat->sprite, x, y,
>  				 es->buffer->width, es->buffer->height);
>  
> +	empty_region(&es->input);
> +
>  	if (!weston_surface_is_mapped(es)) {
>  		wl_list_insert(&es->compositor->cursor_layer.surface_list,
>  			       &es->layer_link);
>  		weston_surface_assign_output(es);
> -		empty_region(&es->input);
>  	}
>  }
>  
> -- 
> 1.7.4.1
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list