xwm: debug changes to override-redirect flag
For every event we handle and that delivers the override-redirect flag,
print it to debug log.
Add a comment to one code path explaining when it gets hit, because it
is unobvious. It also serves as a reminder that we do not handle changes
to the OR flag after Window creation.
Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Reviewed-by: Quentin Glidic <sardemff7+git@sardemff7.net>
diff --git a/xwayland/window-manager.c b/xwayland/window-manager.c
index 5e94bdc..d56eddd 100644
--- a/xwayland/window-manager.c
+++ b/xwayland/window-manager.c
@@ -737,10 +737,11 @@
(xcb_configure_notify_event_t *) event;
struct weston_wm_window *window;
- wm_log("XCB_CONFIGURE_NOTIFY (window %d) %d,%d @ %dx%d\n",
+ wm_log("XCB_CONFIGURE_NOTIFY (window %d) %d,%d @ %dx%d%s\n",
configure_notify->window,
configure_notify->x, configure_notify->y,
- configure_notify->width, configure_notify->height);
+ configure_notify->width, configure_notify->height,
+ configure_notify->override_redirect ? ", override" : "");
if (!wm_lookup_window(wm, configure_notify->window, &window))
return;
@@ -1061,7 +1062,8 @@
return;
}
- wm_log("XCB_MAP_NOTIFY (window %d)\n", map_notify->window);
+ wm_log("XCB_MAP_NOTIFY (window %d%s)\n", map_notify->window,
+ map_notify->override_redirect ? ", override" : "");
}
static void
@@ -1204,6 +1206,11 @@
if (window->frame_id == XCB_WINDOW_NONE) {
if (window->surface != NULL) {
+ /* Override-redirect windows go through here, but we
+ * cannot assert(window->override_redirect); because
+ * we do not deal with changing OR flag yet.
+ * XXX: handle OR flag changes in message handlers
+ */
weston_wm_window_get_frame_size(window, &width, &height);
pixman_region32_fini(&window->surface->pending.opaque);
if (window->has_alpha) {
@@ -1376,10 +1383,11 @@
(xcb_reparent_notify_event_t *) event;
struct weston_wm_window *window;
- wm_log("XCB_REPARENT_NOTIFY (window %d, parent %d, event %d)\n",
+ wm_log("XCB_REPARENT_NOTIFY (window %d, parent %d, event %d%s)\n",
reparent_notify->window,
reparent_notify->parent,
- reparent_notify->event);
+ reparent_notify->event,
+ reparent_notify->override_redirect ? ", override" : "");
if (reparent_notify->parent == wm->screen->root) {
weston_wm_window_create(wm, reparent_notify->window, 10, 10,