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/vo_wlshm.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/vo_wlshm.c')
| -rw-r--r-- | video/out/vo_wlshm.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/video/out/vo_wlshm.c b/video/out/vo_wlshm.c index dbfa386789..f6c37714da 100644 --- a/video/out/vo_wlshm.c +++ b/video/out/vo_wlshm.c @@ -219,10 +219,7 @@ static int resize(struct vo *vo) p->free_buffers = buf->next; talloc_free(buf); } - int ret = mp_sws_reinit(p->sws); - if (!wl->vo_opts->fullscreen && !wl->vo_opts->window_maximized) - wl_surface_commit(wl->surface); - return ret; + return mp_sws_reinit(p->sws); } static int control(struct vo *vo, uint32_t request, void *data) |