From: Rémi Bernon <rbernon@codeweavers.com> --- dlls/win32u/driver.c | 60 +++++++++++++------------- dlls/win32u/opengl.c | 2 +- dlls/win32u/vulkan.c | 2 +- dlls/win32u/win32u_private.h | 1 - dlls/wineandroid.drv/android.h | 1 + dlls/wineandroid.drv/init.c | 1 + dlls/wineandroid.drv/opengl.c | 7 ++- dlls/winemac.drv/gdi.c | 1 + dlls/winemac.drv/macdrv.h | 8 +--- dlls/winemac.drv/opengl.c | 8 ++-- dlls/winemac.drv/vulkan.c | 4 +- dlls/winemac.drv/window.c | 10 ++++- dlls/winewayland.drv/opengl.c | 14 +++--- dlls/winewayland.drv/vulkan.c | 5 ++- dlls/winewayland.drv/wayland_surface.c | 15 ++++--- dlls/winewayland.drv/waylanddrv.h | 4 +- dlls/winewayland.drv/waylanddrv_main.c | 1 + dlls/winex11.drv/init.c | 38 ++++++---------- dlls/winex11.drv/opengl.c | 18 ++++---- dlls/winex11.drv/vulkan.c | 7 ++- dlls/winex11.drv/x11drv.h | 16 ++++++- include/wine/gdi_driver.h | 3 +- 22 files changed, 129 insertions(+), 97 deletions(-) diff --git a/dlls/win32u/driver.c b/dlls/win32u/driver.c index c7a78cd327d..cfe0650648b 100644 --- a/dlls/win32u/driver.c +++ b/dlls/win32u/driver.c @@ -879,6 +879,35 @@ static BOOL nulldrv_GetWindowStateUpdates( HWND hwnd, UINT *state_cmd, UINT *swp return FALSE; } +static void nulldrv_surface_destroy( struct client_surface *client ) +{ +} + +static void nulldrv_surface_detach( struct client_surface *client ) +{ +} + +static void nulldrv_surface_update( struct client_surface *client ) +{ +} + +static void nulldrv_surface_present( struct client_surface *client, HDC hdc ) +{ +} + +static const struct client_surface_funcs nulldrv_surface_funcs = +{ + .destroy = nulldrv_surface_destroy, + .detach = nulldrv_surface_detach, + .update = nulldrv_surface_update, + .present = nulldrv_surface_present, +}; + +static struct client_surface *nulldrv_CreateClientSurface( HWND hwnd, int pixel_format ) +{ + return client_surface_create( sizeof(struct client_surface), &nulldrv_surface_funcs, hwnd ); +} + static BOOL nulldrv_CreateWindowSurface( HWND hwnd, BOOL layered, const RECT *surface_rect, struct window_surface **surface ) { return FALSE; @@ -1302,6 +1331,7 @@ static const struct user_driver_funcs lazy_load_driver = nulldrv_WindowPosChanging, nulldrv_GetWindowStyleMasks, nulldrv_GetWindowStateUpdates, + nulldrv_CreateClientSurface, nulldrv_CreateWindowSurface, nulldrv_MoveWindowBits, nulldrv_WindowPosChanged, @@ -1401,6 +1431,7 @@ void __wine_set_user_driver( const struct user_driver_funcs *funcs, UINT version SET_USER_FUNC(WindowPosChanging); SET_USER_FUNC(GetWindowStyleMasks); SET_USER_FUNC(GetWindowStateUpdates); + SET_USER_FUNC(CreateClientSurface); SET_USER_FUNC(CreateWindowSurface); SET_USER_FUNC(MoveWindowBits); SET_USER_FUNC(WindowPosChanged); @@ -1439,32 +1470,3 @@ INT WINAPI NtGdiExtEscape( HDC hdc, WCHAR *driver, int driver_id, INT escape, IN release_dc_ptr( dc ); return ret; } - -static void nulldrv_surface_destroy( struct client_surface *client ) -{ -} - -static void nulldrv_surface_detach( struct client_surface *client ) -{ -} - -static void nulldrv_surface_update( struct client_surface *client ) -{ -} - -static void nulldrv_surface_present( struct client_surface *client, HDC hdc ) -{ -} - -static const struct client_surface_funcs nulldrv_surface_funcs = -{ - .destroy = nulldrv_surface_destroy, - .detach = nulldrv_surface_detach, - .update = nulldrv_surface_update, - .present = nulldrv_surface_present, -}; - -struct client_surface *nulldrv_client_surface_create( HWND hwnd ) -{ - return client_surface_create( sizeof(struct client_surface), &nulldrv_surface_funcs, hwnd ); -} diff --git a/dlls/win32u/opengl.c b/dlls/win32u/opengl.c index 3d03b41fc24..089452198de 100644 --- a/dlls/win32u/opengl.c +++ b/dlls/win32u/opengl.c @@ -664,7 +664,7 @@ static BOOL egldrv_surface_create( HWND hwnd, int format, struct opengl_drawable { struct client_surface *client; - if (!(client = nulldrv_client_surface_create( hwnd ))) return FALSE; + if (!(client = user_driver->pCreateClientSurface( hwnd, format ))) return FALSE; *drawable = framebuffer_surface_create( format, client ); client_surface_release( client ); diff --git a/dlls/win32u/vulkan.c b/dlls/win32u/vulkan.c index bf8f6f1daba..c95038ed6a4 100644 --- a/dlls/win32u/vulkan.c +++ b/dlls/win32u/vulkan.c @@ -2930,7 +2930,7 @@ static VkResult nulldrv_vulkan_surface_create( HWND hwnd, const struct vulkan_in VkHeadlessSurfaceCreateInfoEXT create_info = {.sType = VK_STRUCTURE_TYPE_HEADLESS_SURFACE_CREATE_INFO_EXT}; VkResult res; - if (!(*client = nulldrv_client_surface_create( hwnd ))) return VK_ERROR_OUT_OF_HOST_MEMORY; + if (!(*client = user_driver->pCreateClientSurface( hwnd, 0 ))) return VK_ERROR_OUT_OF_HOST_MEMORY; if ((res = instance->p_vkCreateHeadlessSurfaceEXT( instance->host.instance, &create_info, NULL, surface ))) { client_surface_release(*client); diff --git a/dlls/win32u/win32u_private.h b/dlls/win32u/win32u_private.h index be303ae4d13..620d848488d 100644 --- a/dlls/win32u/win32u_private.h +++ b/dlls/win32u/win32u_private.h @@ -354,7 +354,6 @@ extern HKEY hkcu_key; /* driver.c */ extern const struct user_driver_funcs *user_driver; -extern struct client_surface *nulldrv_client_surface_create( HWND hwnd ); extern ULONG_PTR zero_bits; diff --git a/dlls/wineandroid.drv/android.h b/dlls/wineandroid.drv/android.h index cc558aa01f6..c63e7b1fd79 100644 --- a/dlls/wineandroid.drv/android.h +++ b/dlls/wineandroid.drv/android.h @@ -115,6 +115,7 @@ extern void ANDROID_SetCapture( HWND hwnd, UINT flags ); extern UINT ANDROID_ShowWindow( HWND hwnd, INT cmd, RECT *rect, UINT swp ); extern LRESULT ANDROID_WindowMessage( HWND hwnd, UINT msg, WPARAM wp, LPARAM lp ); extern BOOL ANDROID_WindowPosChanging( HWND hwnd, UINT swp_flags, BOOL shaped, const struct window_rects *rects ); +extern struct client_surface *ANDROID_CreateClientSurface( HWND hwnd, int pixel_format ); extern BOOL ANDROID_CreateWindowSurface( HWND hwnd, BOOL layered, const RECT *surface_rect, struct window_surface **surface ); extern void ANDROID_WindowPosChanged( HWND hwnd, HWND insert_after, HWND owner_hint, UINT swp_flags, const struct window_rects *new_rects, struct window_surface *surface ); diff --git a/dlls/wineandroid.drv/init.c b/dlls/wineandroid.drv/init.c index df0cb2b72ec..2dca7362a69 100644 --- a/dlls/wineandroid.drv/init.c +++ b/dlls/wineandroid.drv/init.c @@ -322,6 +322,7 @@ static const struct user_driver_funcs android_drv_funcs = .pShowWindow = ANDROID_ShowWindow, .pWindowMessage = ANDROID_WindowMessage, .pWindowPosChanging = ANDROID_WindowPosChanging, + .pCreateClientSurface = ANDROID_CreateClientSurface, .pCreateWindowSurface = ANDROID_CreateWindowSurface, .pWindowPosChanged = ANDROID_WindowPosChanged, .pOpenGLInit = ANDROID_OpenGLInit, diff --git a/dlls/wineandroid.drv/opengl.c b/dlls/wineandroid.drv/opengl.c index 62b4fa7dd6c..2800126761f 100644 --- a/dlls/wineandroid.drv/opengl.c +++ b/dlls/wineandroid.drv/opengl.c @@ -108,7 +108,7 @@ static BOOL android_surface_create( HWND hwnd, int format, struct opengl_drawabl EGLConfig config = egl_config_for_format( format ); struct client_surface *client; - if (!(client = client_surface_create( sizeof(*client), &android_client_surface_funcs, hwnd ))) return FALSE; + if (!(client = ANDROID_CreateClientSurface( hwnd, format ))) return FALSE; gl = opengl_drawable_create( sizeof(*gl), &android_drawable_funcs, format, client ); client_surface_release( client ); if (!gl) return FALSE; @@ -194,6 +194,11 @@ static const struct client_surface_funcs android_client_surface_funcs = .present = android_client_surface_present, }; +struct client_surface *ANDROID_CreateClientSurface( HWND hwnd, int pixel_format ) +{ + return client_surface_create( sizeof(struct client_surface), &android_client_surface_funcs, hwnd ); +} + static const struct opengl_drawable_funcs android_drawable_funcs = { .destroy = android_drawable_destroy, diff --git a/dlls/winemac.drv/gdi.c b/dlls/winemac.drv/gdi.c index a0f314234b9..906bb4d09e6 100644 --- a/dlls/winemac.drv/gdi.c +++ b/dlls/winemac.drv/gdi.c @@ -284,6 +284,7 @@ static const struct user_driver_funcs macdrv_funcs = .pWindowPosChanged = macdrv_WindowPosChanged, .pWindowPosChanging = macdrv_WindowPosChanging, .pGetWindowStyleMasks = macdrv_GetWindowStyleMasks, + .pCreateClientSurface = macdrv_CreateClientSurface, .pCreateWindowSurface = macdrv_CreateWindowSurface, .pVulkanInit = macdrv_VulkanInit, .pOpenGLInit = macdrv_OpenGLInit, diff --git a/dlls/winemac.drv/macdrv.h b/dlls/winemac.drv/macdrv.h index 4f7107baf68..0aef8cdd60f 100644 --- a/dlls/winemac.drv/macdrv.h +++ b/dlls/winemac.drv/macdrv.h @@ -150,6 +150,7 @@ extern void macdrv_SetLayeredWindowAttributes(HWND hwnd, COLORREF key, BYTE alph extern LRESULT macdrv_WindowMessage(HWND hwnd, UINT msg, WPARAM wp, LPARAM lp); extern BOOL macdrv_WindowPosChanging(HWND hwnd, UINT swp_flags, BOOL shaped, const struct window_rects *rects); extern BOOL macdrv_GetWindowStyleMasks(HWND hwnd, UINT style, UINT ex_style, UINT *style_mask, UINT *ex_style_mask); +extern struct client_surface *macdrv_CreateClientSurface(HWND hwnd, int pixel_format); extern BOOL macdrv_CreateWindowSurface(HWND hwnd, BOOL layered, const RECT *surface_rect, struct window_surface **surface); extern void macdrv_WindowPosChanged(HWND hwnd, HWND insert_after, HWND owner_hint, UINT swp_flags, const struct window_rects *new_rects, struct window_surface *surface); @@ -200,12 +201,7 @@ extern BOOL macdrv_SystemParametersInfo(UINT action, UINT int_param, void *ptr_p macdrv_metal_swapchain metal_swapchain; }; -static inline struct macdrv_client_surface *impl_from_client_surface(struct client_surface *client) -{ - return CONTAINING_RECORD(client, struct macdrv_client_surface, client); -} - -extern struct macdrv_client_surface *macdrv_client_surface_create(HWND hwnd); +extern struct macdrv_client_surface *impl_from_client_surface(struct client_surface *client); extern BOOL macdrv_client_surface_acquire_metal_swapchain(struct macdrv_client_surface *surface); extern struct macdrv_win_data *get_win_data(HWND hwnd); diff --git a/dlls/winemac.drv/opengl.c b/dlls/winemac.drv/opengl.c index 0af1d3f7192..86830bffc80 100644 --- a/dlls/winemac.drv/opengl.c +++ b/dlls/winemac.drv/opengl.c @@ -1457,7 +1457,7 @@ static BOOL create_context(struct macdrv_context *context, CGLContextObj share, static BOOL macdrv_surface_create(HWND hwnd, int format, struct opengl_drawable **drawable) { - struct macdrv_client_surface *client; + struct client_surface *client; struct macdrv_win_data *data; struct gl_drawable *gl; @@ -1472,9 +1472,9 @@ static BOOL macdrv_surface_create(HWND hwnd, int format, struct opengl_drawable data->pixel_format = format; release_win_data(data); - if (!(client = macdrv_client_surface_create(hwnd))) return FALSE; - gl = opengl_drawable_create(sizeof(*gl), &macdrv_surface_funcs, format, &client->client); - client_surface_release(&client->client); + if (!(client = macdrv_CreateClientSurface(hwnd, format))) return FALSE; + gl = opengl_drawable_create(sizeof(*gl), &macdrv_surface_funcs, format, client); + client_surface_release(client); if (!gl) return FALSE; *drawable = &gl->base; diff --git a/dlls/winemac.drv/vulkan.c b/dlls/winemac.drv/vulkan.c index f1643e3e5c4..41c148627fb 100644 --- a/dlls/winemac.drv/vulkan.c +++ b/dlls/winemac.drv/vulkan.c @@ -45,10 +45,12 @@ static VkResult macdrv_vulkan_surface_create(HWND hwnd, const struct vulkan_inst { VkResult res; struct macdrv_client_surface *surface; + struct client_surface *ptr; TRACE("%p %p %p %p\n", hwnd, instance, handle, client); - if (!(surface = macdrv_client_surface_create(hwnd))) return VK_ERROR_OUT_OF_HOST_MEMORY; + if (!(ptr = macdrv_CreateClientSurface(hwnd, 0))) return VK_ERROR_OUT_OF_HOST_MEMORY; + surface = impl_from_client_surface( ptr ); if (!macdrv_client_surface_acquire_metal_swapchain(surface)) goto err; diff --git a/dlls/winemac.drv/window.c b/dlls/winemac.drv/window.c index adfc059e34f..4ed488cd760 100644 --- a/dlls/winemac.drv/window.c +++ b/dlls/winemac.drv/window.c @@ -1144,7 +1144,13 @@ static const struct client_surface_funcs macdrv_client_surface_funcs = .present = macdrv_client_surface_present, }; -struct macdrv_client_surface *macdrv_client_surface_create(HWND hwnd) +struct macdrv_client_surface *impl_from_client_surface(struct client_surface *client) +{ + assert(client->funcs == &macdrv_client_surface_funcs); + return CONTAINING_RECORD(client, struct macdrv_client_surface, client); +} + +struct client_surface *macdrv_CreateClientSurface(HWND hwnd, int pixel_format) { HWND toplevel = NtUserGetAncestor(hwnd, GA_ROOT); struct macdrv_client_surface *surface; @@ -1163,7 +1169,7 @@ struct macdrv_client_surface *macdrv_client_surface_create(HWND hwnd) macdrv_client_surface_present(&surface->client, 0); } - return surface; + return &surface->client; } BOOL macdrv_client_surface_acquire_metal_swapchain(struct macdrv_client_surface *surface) diff --git a/dlls/winewayland.drv/opengl.c b/dlls/winewayland.drv/opengl.c index 66457e1aea5..9719c63719d 100644 --- a/dlls/winewayland.drv/opengl.c +++ b/dlls/winewayland.drv/opengl.c @@ -83,10 +83,11 @@ static void wayland_gl_drawable_sync_size(struct wayland_gl_drawable *gl) static BOOL wayland_opengl_surface_create(HWND hwnd, int format, struct opengl_drawable **drawable) { EGLConfig config = egl_config_for_format(format); - struct wayland_client_surface *client; + struct wayland_client_surface *surface; EGLint attribs[4], *attrib = attribs; struct opengl_drawable *previous; struct wayland_gl_drawable *gl; + struct client_surface *client; RECT rect; TRACE("hwnd=%p format=%d\n", hwnd, format); @@ -106,9 +107,10 @@ static BOOL wayland_opengl_surface_create(HWND hwnd, int format, struct opengl_d } *attrib++ = EGL_NONE; - if (!(client = wayland_client_surface_create(hwnd))) return FALSE; - gl = opengl_drawable_create(sizeof(*gl), &wayland_drawable_funcs, format, &client->client); - client_surface_release(&client->client); + if (!(client = WAYLAND_CreateClientSurface(hwnd, format))) return FALSE; + gl = opengl_drawable_create(sizeof(*gl), &wayland_drawable_funcs, format, client); + surface = impl_from_client_surface(client); /* reference held by gl */ + client_surface_release(client); if (!gl) return FALSE; opengl_drawable_map_buffer(&gl->base, GL_FRONT_LEFT, GL_BACK_LEFT); @@ -116,9 +118,9 @@ static BOOL wayland_opengl_surface_create(HWND hwnd, int format, struct opengl_d opengl_drawable_map_buffer(&gl->base, GL_FRONT_AND_BACK, GL_BACK); if (gl->base.stereo) opengl_drawable_map_buffer(&gl->base, GL_FRONT_RIGHT, GL_BACK_RIGHT); - if (!(gl->wl_egl_window = wl_egl_window_create(client->wl_surface, rect.right, rect.bottom))) goto err; + if (!(gl->wl_egl_window = wl_egl_window_create(surface->wl_surface, rect.right, rect.bottom))) goto err; if (!(gl->base.surface = funcs->p_eglCreateWindowSurface(egl->display, config, gl->wl_egl_window, attribs))) goto err; - set_client_surface(hwnd, client); + set_client_surface(hwnd, surface); TRACE("Created drawable %s with egl_surface %p\n", debugstr_opengl_drawable(&gl->base), gl->base.surface); diff --git a/dlls/winewayland.drv/vulkan.c b/dlls/winewayland.drv/vulkan.c index b2e91102f58..65ebbc40b1f 100644 --- a/dlls/winewayland.drv/vulkan.c +++ b/dlls/winewayland.drv/vulkan.c @@ -44,10 +44,13 @@ static VkResult wayland_vulkan_surface_create(HWND hwnd, const struct vulkan_ins VkResult res; VkWaylandSurfaceCreateInfoKHR create_info_host; struct wayland_client_surface *surface; + struct client_surface *ptr; TRACE("%p %p %p %p\n", hwnd, instance, handle, client); - if (!(surface = wayland_client_surface_create(hwnd))) return VK_ERROR_OUT_OF_HOST_MEMORY; + if (!(ptr = WAYLAND_CreateClientSurface(hwnd, 0))) return VK_ERROR_OUT_OF_HOST_MEMORY; + surface = impl_from_client_surface( ptr ); + create_info_host.sType = VK_STRUCTURE_TYPE_WAYLAND_SURFACE_CREATE_INFO_KHR; create_info_host.pNext = NULL; create_info_host.flags = 0; /* reserved */ diff --git a/dlls/winewayland.drv/wayland_surface.c b/dlls/winewayland.drv/wayland_surface.c index beb7f5734a7..b28072331d6 100644 --- a/dlls/winewayland.drv/wayland_surface.c +++ b/dlls/winewayland.drv/wayland_surface.c @@ -1149,11 +1149,6 @@ void wayland_surface_coords_to_window(struct wayland_surface *surface, *window_y = round(surface_y * surface->window.scale); } -static struct wayland_client_surface *impl_from_client_surface(struct client_surface *client) -{ - return CONTAINING_RECORD(client, struct wayland_client_surface, client); -} - static void wayland_client_surface_destroy(struct client_surface *client) { struct wayland_client_surface *surface = impl_from_client_surface(client); @@ -1213,7 +1208,13 @@ static const struct client_surface_funcs wayland_client_surface_funcs = .present = wayland_client_surface_present, }; -struct wayland_client_surface *wayland_client_surface_create(HWND hwnd) +struct wayland_client_surface *impl_from_client_surface(struct client_surface *client) +{ + assert(client->funcs == &wayland_client_surface_funcs); + return CONTAINING_RECORD(client, struct wayland_client_surface, client); +} + +struct client_surface *WAYLAND_CreateClientSurface(HWND hwnd, int pixel_format) { struct wayland_client_surface *client; struct wl_region *empty_region; @@ -1248,7 +1249,7 @@ struct wayland_client_surface *wayland_client_surface_create(HWND hwnd) goto err; } - return client; + return &client->client; err: client_surface_release(&client->client); diff --git a/dlls/winewayland.drv/waylanddrv.h b/dlls/winewayland.drv/waylanddrv.h index 3d30ab3d59c..07ac10e9f9e 100644 --- a/dlls/winewayland.drv/waylanddrv.h +++ b/dlls/winewayland.drv/waylanddrv.h @@ -254,6 +254,8 @@ struct wayland_client_surface struct wp_viewport *wp_viewport; }; +extern struct wayland_client_surface *impl_from_client_surface(struct client_surface *client); + struct wayland_shm_buffer { struct wl_list link; @@ -338,7 +340,6 @@ void wayland_surface_coords_from_window(struct wayland_surface *surface, void wayland_surface_coords_to_window(struct wayland_surface *surface, double surface_x, double surface_y, int *window_x, int *window_y); -struct wayland_client_surface *wayland_client_surface_create(HWND hwnd); void wayland_client_surface_attach(struct wayland_client_surface *client, HWND toplevel); void wayland_surface_ensure_contents(struct wayland_surface *surface); void wayland_surface_set_title(struct wayland_surface *surface, LPCWSTR title); @@ -469,6 +470,7 @@ LRESULT WAYLAND_WindowMessage(HWND hwnd, UINT msg, WPARAM wp, LPARAM lp); void WAYLAND_WindowPosChanged(HWND hwnd, HWND insert_after, HWND owner_hint, UINT swp_flags, const struct window_rects *new_rects, struct window_surface *surface); BOOL WAYLAND_WindowPosChanging(HWND hwnd, UINT swp_flags, BOOL shaped, const struct window_rects *rects); +struct client_surface *WAYLAND_CreateClientSurface(HWND hwnd, int pixel_format); BOOL WAYLAND_CreateWindowSurface(HWND hwnd, BOOL layered, const RECT *surface_rect, struct window_surface **surface); UINT WAYLAND_VulkanInit(UINT version, void *vulkan_handle, const struct vulkan_driver_funcs **driver_funcs); UINT WAYLAND_OpenGLInit(UINT version, const struct opengl_funcs *opengl_funcs, const struct opengl_driver_funcs **driver_funcs); diff --git a/dlls/winewayland.drv/waylanddrv_main.c b/dlls/winewayland.drv/waylanddrv_main.c index 5fae15c81eb..275a3907f92 100644 --- a/dlls/winewayland.drv/waylanddrv_main.c +++ b/dlls/winewayland.drv/waylanddrv_main.c @@ -53,6 +53,7 @@ static const struct user_driver_funcs waylanddrv_funcs = .pWindowMessage = WAYLAND_WindowMessage, .pWindowPosChanged = WAYLAND_WindowPosChanged, .pWindowPosChanging = WAYLAND_WindowPosChanging, + .pCreateClientSurface = WAYLAND_CreateClientSurface, .pCreateWindowSurface = WAYLAND_CreateWindowSurface, .pVulkanInit = WAYLAND_VulkanInit, .pOpenGLInit = WAYLAND_OpenGLInit, diff --git a/dlls/winex11.drv/init.c b/dlls/winex11.drv/init.c index 09ef2bf8664..d5b4fdbf0aa 100644 --- a/dlls/winex11.drv/init.c +++ b/dlls/winex11.drv/init.c @@ -26,6 +26,7 @@ #include <stdarg.h> #include <string.h> +#include <assert.h> #include "windef.h" #include "winbase.h" @@ -264,25 +265,6 @@ HRGN get_dc_monitor_region( HWND hwnd, HDC hdc ) return 0; } -static const struct client_surface_funcs x11drv_client_surface_funcs; - -struct x11drv_client_surface -{ - struct client_surface client; - XWindowChanges changes; - Colormap colormap; - Window window; - RECT rect; - - HDC hdc_src; - HDC hdc_dst; -}; - -static struct x11drv_client_surface *impl_from_client_surface( struct client_surface *client ) -{ - return CONTAINING_RECORD( client, struct x11drv_client_surface, client ); -} - static void x11drv_client_surface_destroy( struct client_surface *client ) { struct x11drv_client_surface *surface = impl_from_client_surface( client ); @@ -477,17 +459,23 @@ static int visual_class_alloc( int class ) return class == PseudoColor || class == GrayScale || class == DirectColor ? AllocAll : AllocNone; } -Window x11drv_client_surface_create( HWND hwnd, int format, struct client_surface **client ) +struct x11drv_client_surface *impl_from_client_surface( struct client_surface *client ) +{ + assert( client->funcs == &x11drv_client_surface_funcs ); + return CONTAINING_RECORD( client, struct x11drv_client_surface, client ); +} + +struct client_surface *X11DRV_CreateClientSurface( HWND hwnd, int format ) { struct x11drv_client_surface *surface; XVisualInfo visual = default_visual; Colormap colormap; - if (format && !visual_from_pixel_format( format, &visual )) return None; + if (format && !visual_from_pixel_format( format, &visual )) return NULL; if (visual.visualid == default_visual.visualid) colormap = default_colormap; else colormap = XCreateColormap( gdi_display, get_dummy_parent(), visual.visual, visual_class_alloc( visual.class ) ); - if (!colormap) return None; + if (!colormap) return NULL; if (!(surface = client_surface_create( sizeof(*surface), &x11drv_client_surface_funcs, hwnd ))) goto failed; surface->colormap = colormap; @@ -496,13 +484,12 @@ Window x11drv_client_surface_create( HWND hwnd, int format, struct client_surfac if (!(surface->window = create_client_window( hwnd, surface->rect, &visual, colormap ))) goto failed; TRACE( "Created %s for client window %lx\n", debugstr_client_surface( &surface->client ), surface->window ); - *client = &surface->client; - return surface->window; + return &surface->client; failed: if (surface) client_surface_release( &surface->client ); else if (colormap != default_colormap) XFreeColormap( gdi_display, colormap ); - return None; + return NULL; } /********************************************************************** @@ -711,6 +698,7 @@ static const struct user_driver_funcs x11drv_funcs = .pWindowPosChanging = X11DRV_WindowPosChanging, .pGetWindowStyleMasks = X11DRV_GetWindowStyleMasks, .pGetWindowStateUpdates = X11DRV_GetWindowStateUpdates, + .pCreateClientSurface = X11DRV_CreateClientSurface, .pCreateWindowSurface = X11DRV_CreateWindowSurface, .pMoveWindowBits = X11DRV_MoveWindowBits, .pWindowPosChanged = X11DRV_WindowPosChanged, diff --git a/dlls/winex11.drv/opengl.c b/dlls/winex11.drv/opengl.c index 56c22ab7ee7..2b6e1145e47 100644 --- a/dlls/winex11.drv/opengl.c +++ b/dlls/winex11.drv/opengl.c @@ -521,14 +521,15 @@ static BOOL x11drv_egl_describe_pixel_format( int format, struct wgl_pixel_forma static BOOL x11drv_egl_surface_create( HWND hwnd, int format, struct opengl_drawable **drawable ) { + struct x11drv_client_surface *surface; struct opengl_drawable *previous; struct client_surface *client; struct gl_drawable *gl; - Window window; if ((previous = *drawable) && previous->format == format) return TRUE; - if (!(window = x11drv_client_surface_create( hwnd, format, &client ))) return FALSE; + if (!(client = X11DRV_CreateClientSurface( hwnd, format ))) return FALSE; gl = opengl_drawable_create( sizeof(*gl), &x11drv_egl_surface_funcs, format, client ); + surface = impl_from_client_surface( client ); /* reference held by gl */ client_surface_release( client ); if (!gl) return FALSE; @@ -538,13 +539,13 @@ static BOOL x11drv_egl_surface_create( HWND hwnd, int format, struct opengl_draw if (gl->base.stereo) opengl_drawable_map_buffer( &gl->base, GL_FRONT_RIGHT, GL_BACK_RIGHT ); if (!(gl->base.surface = funcs->p_eglCreateWindowSurface( egl->display, egl_config_for_format( format ), - (void *)window, NULL ))) + (void *)surface->window, NULL ))) { opengl_drawable_release( &gl->base ); return FALSE; } - TRACE( "Created drawable %s with client window %lx\n", debugstr_opengl_drawable( &gl->base ), window ); + TRACE( "Created drawable %s with client window %lx\n", debugstr_opengl_drawable( &gl->base ), surface->window ); XFlush( gdi_display ); if (previous) opengl_drawable_release( previous ); @@ -959,24 +960,25 @@ static GLXContext create_glxcontext( int format, GLXContext share, const int *at static BOOL x11drv_surface_create( HWND hwnd, int format, struct opengl_drawable **drawable ) { struct glx_pixel_format *fmt = glx_pixel_format_from_format( format ); + struct x11drv_client_surface *surface; struct opengl_drawable *previous; struct client_surface *client; struct gl_drawable *gl; - Window window; if ((previous = *drawable) && previous->format == format) return TRUE; - if (!(window = x11drv_client_surface_create( hwnd, format, &client ))) return FALSE; + if (!(client = X11DRV_CreateClientSurface( hwnd, format ))) return FALSE; gl = opengl_drawable_create( sizeof(*gl), &x11drv_surface_funcs, format, client ); + surface = impl_from_client_surface( client ); /* reference held by gl */ client_surface_release( client ); if (!gl) return FALSE; - if (!(gl->drawable = pglXCreateWindow( gdi_display, fmt->fbconfig, window, NULL ))) + if (!(gl->drawable = pglXCreateWindow( gdi_display, fmt->fbconfig, surface->window, NULL ))) { opengl_drawable_release( &gl->base ); return FALSE; } - TRACE( "Created drawable %s with client window %lx\n", debugstr_opengl_drawable( &gl->base ), window ); + TRACE( "Created drawable %s with client window %lx\n", debugstr_opengl_drawable( &gl->base ), surface->window ); XFlush( gdi_display ); if (previous) opengl_drawable_release( previous ); diff --git a/dlls/winex11.drv/vulkan.c b/dlls/winex11.drv/vulkan.c index e11a989f02d..fd58d4662d7 100644 --- a/dlls/winex11.drv/vulkan.c +++ b/dlls/winex11.drv/vulkan.c @@ -54,10 +54,15 @@ static VkResult X11DRV_vulkan_surface_create( HWND hwnd, const struct vulkan_ins .sType = VK_STRUCTURE_TYPE_XLIB_SURFACE_CREATE_INFO_KHR, .dpy = gdi_display, }; + struct x11drv_client_surface *surface; + struct client_surface *ptr; TRACE( "%p %p %p %p\n", hwnd, instance, handle, client ); - if (!(info.window = x11drv_client_surface_create( hwnd, 0, client ))) return VK_ERROR_OUT_OF_HOST_MEMORY; + if (!(ptr = X11DRV_CreateClientSurface( hwnd, 0 ))) return VK_ERROR_OUT_OF_HOST_MEMORY; + surface = impl_from_client_surface( ptr ); + + info.window = surface->window; if (instance->p_vkCreateXlibSurfaceKHR( instance->host.instance, &info, NULL /* allocator */, handle )) { ERR("Failed to create Xlib surface\n"); diff --git a/dlls/winex11.drv/x11drv.h b/dlls/winex11.drv/x11drv.h index 3def37a8f21..56d6c814aad 100644 --- a/dlls/winex11.drv/x11drv.h +++ b/dlls/winex11.drv/x11drv.h @@ -251,6 +251,7 @@ extern LRESULT X11DRV_WindowMessage( HWND hwnd, UINT msg, WPARAM wp, LPARAM lp ) extern BOOL X11DRV_WindowPosChanging( HWND hwnd, UINT swp_flags, BOOL shaped, const struct window_rects *rects ); extern BOOL X11DRV_GetWindowStyleMasks( HWND hwnd, UINT style, UINT ex_style, UINT *style_mask, UINT *ex_style_mask ); extern BOOL X11DRV_GetWindowStateUpdates( HWND hwnd, UINT *state_cmd, UINT *swp_flags, RECT *rect, HWND *foreground ); +extern struct client_surface *X11DRV_CreateClientSurface( HWND hwnd, int format ); extern BOOL X11DRV_CreateWindowSurface( HWND hwnd, BOOL layered, const RECT *surface_rect, struct window_surface **surface ); extern void X11DRV_MoveWindowBits( HWND hwnd, const struct window_rects *old_rects, const struct window_rects *new_rects, const RECT *valid_rects ); @@ -364,11 +365,24 @@ struct x11drv_escape_get_drawable RECT dc_rect; /* DC rectangle relative to drawable */ }; +struct x11drv_client_surface +{ + struct client_surface client; + XWindowChanges changes; + Colormap colormap; + Window window; + RECT rect; + + HDC hdc_src; + HDC hdc_dst; +}; + +extern struct x11drv_client_surface *impl_from_client_surface( struct client_surface *client ); + extern BOOL needs_offscreen_rendering( HWND hwnd ); extern void set_dc_drawable( HDC hdc, Drawable drawable, const RECT *rect, int mode ); extern Drawable get_dc_drawable( HDC hdc, RECT *rect ); extern HRGN get_dc_monitor_region( HWND hwnd, HDC hdc ); -extern Window x11drv_client_surface_create( HWND hwnd, int format, struct client_surface **client ); /************************************************************************** * X11 USER driver diff --git a/include/wine/gdi_driver.h b/include/wine/gdi_driver.h index 97c646d6ec9..fa1f7232505 100644 --- a/include/wine/gdi_driver.h +++ b/include/wine/gdi_driver.h @@ -218,7 +218,7 @@ struct gdi_dc_funcs }; /* increment this when you change the DC function table */ -#define WINE_GDI_DRIVER_VERSION 108 +#define WINE_GDI_DRIVER_VERSION 109 #define GDI_PRIORITY_NULL_DRV 0 /* null driver */ #define GDI_PRIORITY_FONT_DRV 100 /* any font driver */ @@ -420,6 +420,7 @@ struct user_driver_funcs BOOL (*pWindowPosChanging)(HWND,UINT,BOOL,const struct window_rects *); BOOL (*pGetWindowStyleMasks)(HWND,UINT,UINT,UINT*,UINT*); BOOL (*pGetWindowStateUpdates)(HWND,UINT*,UINT*,RECT*,HWND*); + struct client_surface *(*pCreateClientSurface)(HWND,int); BOOL (*pCreateWindowSurface)(HWND,BOOL,const RECT *,struct window_surface**); void (*pMoveWindowBits)(HWND,const struct window_rects *,const struct window_rects *,const RECT *); void (*pWindowPosChanged)(HWND,HWND,HWND,UINT,const struct window_rects*,struct window_surface*); -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/11162