diff options
| author | Dudemanguy <random342@airmail.cc> | 2020-11-08 09:51:52 -0600 |
|---|---|---|
| committer | Dudemanguy <random342@airmail.cc> | 2020-11-08 09:51:52 -0600 |
| commit | dae6b1be967b6767409005e6edc7a8021fd98fe8 (patch) | |
| tree | 5bf9a898946b1811d4fdcf26299a72476adccd09 /video/out/vulkan/context_wayland.c | |
| parent | 790647314ca85cd224272638a73780b0bdfd1deb (diff) | |
| download | mpv-dae6b1be967b6767409005e6edc7a8021fd98fe8.tar.bz2 mpv-dae6b1be967b6767409005e6edc7a8021fd98fe8.tar.xz | |
Revert "wayland: conditionally commit surface on resize"
30dcfbc is a workaround for incorrect border sizes that could occur on
sway/wlroots in certain edge cases. This seemed harmless enough, but it
turns out that on mutter the extra wl_surface_commit somehow causes the
window always go to the top left of the screen after you leave
fullscreen. No idea why this occurs, but the original commit is a
workaround a sway bug and causing regressions for other users isn't
right despite the author being biased towards sway/wlroots.
This reverts commit 30dcfbc9cb3f77dbb729fb6f95ffde7dbdddc4cb.
Diffstat (limited to 'video/out/vulkan/context_wayland.c')
| -rw-r--r-- | video/out/vulkan/context_wayland.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/video/out/vulkan/context_wayland.c b/video/out/vulkan/context_wayland.c index 7e6beb3bdd..98bc8afcfc 100644 --- a/video/out/vulkan/context_wayland.c +++ b/video/out/vulkan/context_wayland.c @@ -210,10 +210,7 @@ static bool resize(struct ra_ctx *ctx) vo_wayland_set_opaque_region(wl, ctx->opts.want_alpha); wl_surface_set_buffer_scale(wl->surface, wl->scaling); - bool ok = ra_vk_ctx_resize(ctx, width, height); - if (!wl->vo_opts->fullscreen && !wl->vo_opts->window_maximized) - wl_surface_commit(wl->surface); - return ok; + return ra_vk_ctx_resize(ctx, width, height); } static bool wayland_vk_reconfig(struct ra_ctx *ctx) |