xwm: fix initially-fullscreen windows

Graphics / Wayland / Weston - Pekka Paalanen [collabora.com] - 29 January 2020 11:31 EST

It looks like commit ad0da4596d6315b18e888af75eee0a9bad1ff44d introduced a bug for X11 windows that are initially fullscreen by adding code to the end of xserver_map_shell_surface() while ignoring the 'return' that this patch removes. That may have caused some annoying window state issues, but the problem became more pronounced with 7ace831ca6205ea288e49fdbd6b63f53e73fae59 when used with an Xwayland version that honours _XWAYLAND_ALLOW_COMMITS.

In the latter case, there is a possiblity the window will never show up, as XWM forgets to set allow_commits=true. However, the window may sometimes actually show up due to an oversight in Xwayland: the Present code may be flipping the window buffers and not checking _XWAYLAND_ALLOW_COMMITS if it is supposed commit at all.

Since then, f568968f8a30eab6bfd8a15518014deb8f6c81d5 added more places where allow_commits is set to true, masking the window-does-not-show-up issue. Window pending state likely still remained an issue.

This patch properly fixes the "window never appears" issue by making sure allow_commit=true is set. At the same time, it ensures the pending state functions are called at the end of xserver_map_shell_surface(), which may fix some window state issues like misplaced decorations and/or position of initially-fullscreen windows. Unfortunately, it certainly does not fix all such state problems.

134e1496 xwm: fix initially-fullscreen windows
xwayland/window-manager.c | 1 -
1 file changed, 1 deletion(-)

Upstream: cgit.freedesktop.org

  • Share