Skip to content

W
wine-winehq
Commit ab3d6d98 authored by Alexandros Frantzis's avatar Alexandros Frantzis Committed by Alexandre Julliard
Browse files
Options

winewayland.drv: Handle application-initiated fullscreen state.

Detect when an window wants to become fullscreen and request the Wayland fullscreen state from the compositor. The fullscreen state for windows is inferred by their size and position relative to the monitor that contains their largest region. Since there is no explicit fullscreen state for windows, compositor initiated fullscreen requests may lead to a size change, but will not directly affect other aspects of the window state.
parent 8f0d3ec1
Hide whitespace changes
Inline Side-by-side
Showing with 64 additions and 14 deletions
dlls/winewayland.drv/wayland_surface.c
View file @ ab3d6d98
......@@ -97,6 +97,9 @@ static void xdg_toplevel_handle_configure(void *data,
case XDG_TOPLEVEL_STATE_TILED_BOTTOM:
config_state |= WAYLAND_SURFACE_CONFIG_STATE_TILED;
break;
case XDG_TOPLEVEL_STATE_FULLSCREEN:
config_state |= WAYLAND_SURFACE_CONFIG_STATE_FULLSCREEN;
break;
default:
break;
}
......@@ -338,6 +341,13 @@ static BOOL wayland_surface_configure_is_compatible(struct wayland_surface_confi
return FALSE;
}
/* The fullscreen state requires at most the configured size. */
if ((conf->state & WAYLAND_SURFACE_CONFIG_STATE_FULLSCREEN) &&
(width > conf->width || height > conf->height))
{
return FALSE;
}
return TRUE;
}
......
dlls/winewayland.drv/waylanddrv.h
View file @ ab3d6d98
......@@ -61,7 +61,8 @@ enum wayland_surface_config_state
{
WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED = (1 << 0),
WAYLAND_SURFACE_CONFIG_STATE_RESIZING = (1 << 1),
WAYLAND_SURFACE_CONFIG_STATE_TILED = (1 << 2)
WAYLAND_SURFACE_CONFIG_STATE_TILED = (1 << 2),
WAYLAND_SURFACE_CONFIG_STATE_FULLSCREEN = (1 << 3)
};
struct wayland_cursor
......
dlls/winewayland.drv/window.c
View file @ ab3d6d98
......@@ -153,9 +153,28 @@ static void wayland_win_data_release(struct wayland_win_data *data)
static void wayland_win_data_get_config(struct wayland_win_data *data,
struct wayland_window_config *conf)
{
enum wayland_surface_config_state window_state = 0;
DWORD style;
conf->rect = data->window_rect;
conf->state = (NtUserGetWindowLongW(data->hwnd, GWL_STYLE) & WS_MAXIMIZE) ?
WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED : 0;
style = NtUserGetWindowLongW(data->hwnd, GWL_STYLE);
TRACE("window=%s style=%#lx\n", wine_dbgstr_rect(&conf->rect), (long)style);
/* The fullscreen state is implied by the window position and style. */
if (NtUserIsWindowRectFullScreen(&conf->rect))
{
if ((style & WS_MAXIMIZE) && (style & WS_CAPTION) == WS_CAPTION)
window_state |= WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED;
else if (!(style & WS_MINIMIZE))
window_state |= WAYLAND_SURFACE_CONFIG_STATE_FULLSCREEN;
}
else if (style & WS_MAXIMIZE)
{
window_state |= WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED;
}
conf->state = window_state;
}
static void wayland_win_data_update_wayland_surface(struct wayland_win_data *data)
......@@ -227,15 +246,28 @@ static void wayland_win_data_update_wayland_state(struct wayland_win_data *data)
* window state to determine and update the Wayland state. */
if (!processing_config)
{
/* First do all state unsettings, before setting new state. Some
* Wayland compositors misbehave if the order is reversed. */
if (!(surface->window.state & WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED) &&
(surface->current.state & WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED))
{
xdg_toplevel_unset_maximized(surface->xdg_toplevel);
}
if (!(surface->window.state & WAYLAND_SURFACE_CONFIG_STATE_FULLSCREEN) &&
(surface->current.state & WAYLAND_SURFACE_CONFIG_STATE_FULLSCREEN))
{
xdg_toplevel_unset_fullscreen(surface->xdg_toplevel);
}
if ((surface->window.state & WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED) &&
!(surface->current.state & WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED))
{
xdg_toplevel_set_maximized(surface->xdg_toplevel);
}
else if (!(surface->window.state & WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED) &&
(surface->current.state & WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED))
if ((surface->window.state & WAYLAND_SURFACE_CONFIG_STATE_FULLSCREEN) &&
!(surface->current.state & WAYLAND_SURFACE_CONFIG_STATE_FULLSCREEN))
{
xdg_toplevel_unset_maximized(surface->xdg_toplevel);
xdg_toplevel_set_fullscreen(surface->xdg_toplevel, NULL);
}
}
else
......@@ -353,7 +385,7 @@ static void wayland_configure_window(HWND hwnd)
{
struct wayland_surface *surface;
INT width, height;
UINT flags;
UINT flags = 0;
uint32_t state;
DWORD style;
BOOL needs_enter_size_move = FALSE;
......@@ -394,6 +426,14 @@ static void wayland_configure_window(HWND hwnd)
needs_exit_size_move = TRUE;
}
/* Transitions between normal/max/fullscreen may entail a frame change. */
if ((state ^ surface->current.state) &
(WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED |
WAYLAND_SURFACE_CONFIG_STATE_FULLSCREEN))
{
flags |= SWP_FRAMECHANGED;
}
pthread_mutex_unlock(&surface->mutex);
TRACE("processing=%dx%d,%#x\n", width, height, state);
......@@ -401,20 +441,19 @@ static void wayland_configure_window(HWND hwnd)
if (needs_enter_size_move) send_message(hwnd, WM_ENTERSIZEMOVE, 0, 0);
if (needs_exit_size_move) send_message(hwnd, WM_EXITSIZEMOVE, 0, 0);
flags = SWP_NOACTIVATE | SWP_NOZORDER | SWP_NOOWNERZORDER | SWP_NOMOVE;
flags |= SWP_NOACTIVATE | SWP_NOZORDER | SWP_NOOWNERZORDER | SWP_NOMOVE;
if (width == 0 || height == 0) flags |= SWP_NOSIZE;
style = NtUserGetWindowLongW(hwnd, GWL_STYLE);
if (!(state & WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED) != !(style & WS_MAXIMIZE))
{
NtUserSetWindowLong(hwnd, GWL_STYLE, style ^ WS_MAXIMIZE, FALSE);
flags |= SWP_FRAMECHANGED;
}
/* The Wayland maximized state is very strict about surface size, so don't
* let the application override it. The tiled state is not as strict,
* but it indicates a strong size preference, so try to respect it. */
/* The Wayland maximized and fullscreen states are very strict about
* surface size, so don't let the application override it. The tiled state
* is not as strict, but it indicates a strong size preference, so try to
* respect it. */
if (state & (WAYLAND_SURFACE_CONFIG_STATE_MAXIMIZED |
WAYLAND_SURFACE_CONFIG_STATE_FULLSCREEN |
WAYLAND_SURFACE_CONFIG_STATE_TILED))
{
flags |= SWP_NOSENDCHANGING;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment