MetaWindowWayland: Don't set X11 window attributes that'll get ignored

We only use a handful of the attributes set, so lets stop pretending
that things are initialized for a reason. Eventually we should stop
using XWindowAttributes in the generic MetaWindow creation path.

https://bugzilla.gnome.org/show_bug.cgi?id=769070
This commit is contained in:
Jonas Ådahl 2016-07-22 16:04:47 +08:00
parent e6972924d0
commit 98cd8136ca

View File

@ -459,33 +459,22 @@ MetaWindow *
meta_window_wayland_new (MetaDisplay *display, meta_window_wayland_new (MetaDisplay *display,
MetaWaylandSurface *surface) MetaWaylandSurface *surface)
{ {
XWindowAttributes attrs; XWindowAttributes attrs = { 0 };
MetaScreen *scr = display->screen; MetaScreen *scr = display->screen;
MetaWindow *window; MetaWindow *window;
/*
* Set attributes used by _meta_window_shared_new, don't bother trying to fake
* X11 window attributes with the rest, since they'll be ignored anyway.
*/
attrs.x = 0; attrs.x = 0;
attrs.y = 0; attrs.y = 0;
attrs.width = 0; attrs.width = 0;
attrs.height = 0; attrs.height = 0;
attrs.border_width = 0;
attrs.depth = 24; attrs.depth = 24;
attrs.visual = NULL; attrs.visual = NULL;
attrs.root = scr->xroot;
attrs.class = InputOutput;
attrs.bit_gravity = NorthWestGravity;
attrs.win_gravity = NorthWestGravity;
attrs.backing_store = 0;
attrs.backing_planes = ~0;
attrs.backing_pixel = 0;
attrs.save_under = 0;
attrs.colormap = 0;
attrs.map_installed = 1;
attrs.map_state = IsUnmapped; attrs.map_state = IsUnmapped;
attrs.all_event_masks = ~0; attrs.override_redirect = False;
attrs.your_event_mask = 0;
attrs.do_not_propagate_mask = 0;
attrs.override_redirect = 0;
attrs.screen = scr->xscreen;
/* XXX: Note: In the Wayland case we currently still trap X errors while /* XXX: Note: In the Wayland case we currently still trap X errors while
* creating a MetaWindow because we will still be making various redundant * creating a MetaWindow because we will still be making various redundant