On Wed Sep 18 08:48:13 2024 +0000, Alexandros Frantzis wrote:
Since this MR is now moving all client surface reconfiguration to the window thread, the chances of unfortunate interactions with the (possibly different) render thread (as discussed above) are increased. So, I would propose removing the `wl_surface_commit(client->wl_surface)` from `wayland_surface_reconfigure_client()`. The trade-off is that the client surface viewport will be applied at render time, which may increase sizing related artifacts when resizing and the GL/VK redraw is slow. With the above, if an application resizes but doesn't render again the viewport will not be applied, but I think that's quite an edge case.
I added a change to remove the client surface commit from our side.