On Mon, Jun 04, 2012 at 08:01:46PM +0300, Tiago Vignatti wrote:
> and read later whether some property like MOTIF_WM_HINTS wants to change it
> to a different value.
> 
> Signed-off-by: Tiago Vignatti <[email protected]>
> ---
>  src/xwayland/window-manager.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/xwayland/window-manager.c b/src/xwayland/window-manager.c
> index b5d39d3..1e121ee 100644
> --- a/src/xwayland/window-manager.c
> +++ b/src/xwayland/window-manager.c
> @@ -252,7 +252,6 @@ weston_wm_window_read_properties(struct weston_wm_window 
> *window)
>                                            props[i].atom,
>                                            XCB_ATOM_ANY, 0, 2048);
>  
> -     window->decorate = 1;
>       for (i = 0; i < ARRAY_LENGTH(props); i++)  {
>               reply = xcb_get_property_reply(wm->conn, cookie[i], NULL);
>               if (!reply)

We need to initialize window->decorate here always.  The *absence* of
the motif hints mean that we have to decorate, so to be able
transition from "not decorated" to "decorated" we need to initialize
window->decorate here.

Kristian

> @@ -712,6 +711,7 @@ weston_wm_window_create(struct weston_wm *wm,
>       window->id = id;
>       window->properties_dirty = 1;
>       window->override_redirect = override;
> +     window->decorate = !window->override_redirect;
>       window->width = width;
>       window->height = height;
>  
> -- 
> 1.7.9.5
> 
> _______________________________________________
> wayland-devel mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to