godotengine
/
godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Wayland: Use suspended event only if supported 

In a previous PR I made an oversight and checked for the new `suspended`
event even on legacy compositors, breaking suspension for them.

This patch goes trough the trouble of checking for the xdg_toplevel
version and seeing whether we have to use one method or the other.
This commit is contained in:
Riteo 2024-09-26 10:07:53 +02:00
parent f7c567e2f5
commit 5ef07b9ce9
1 changed files with 21 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
platform/linuxbsd/wayland/display_server_wayland.cpp
View File

@ -1219,9 +1219,27 @@ void DisplayServerWayland::process_events() {
} else {
try_suspend();
}
} else if (!wayland_thread.is_suspended() || wayland_thread.get_reset_frame()) {
// At last, a sign of life! We're no longer suspended.
suspended = false;
} else {
// We have to pick into the Wayland thread to figure out if suspension events
// are supported or if we have to use an heuristic as a fallback.
WaylandThread::WindowState *ws = wayland_thread.wl_surface_get_window_state(wayland_thread.window_get_wl_surface(MAIN_WINDOW_ID));
ERR_FAIL_NULL(ws);
struct xdg_toplevel *xdg_toplevel = ws->xdg_toplevel;
if (!xdg_toplevel) {
xdg_toplevel = libdecor_frame_get_xdg_toplevel(ws->libdecor_frame);
}
ERR_FAIL_NULL(xdg_toplevel);
bool suspend_supported = xdg_toplevel_get_version(xdg_toplevel) >= 6;
if (suspend_supported) {
suspended = wayland_thread.is_suspended();
} else {
// Older compositors don't have any way of telling us that a window is
// suspended so we'll kick us out of the suspended state with a frame event.
suspended = !wayland_thread.get_reset_frame();
}
}
#ifdef DBUS_ENABLED