winex11: Set res_class to the program name for Crostini.

GNOME implements the startup notification protocol correctly which means it checks StartupWMClass against both WM name (res_name) and WM class (res_class). Thus it does not need this patch. The situation is different for desktop environments that thunk to Wayland such as Crostini. Wayland does not have separate concepts that WM name and WM class can be mapped to. So Crostini decided to only use res_class resulting in it trying to match 'Wine' to the program name stored in StartupWMClass. While Crostini's choice is unfortunate for Wine, most other applications (e.g. all GTK applications) already store the same value in both WM name and class. So in the interest of compatiblity it makes sense for Wine to do the same. Signed-off-by: Francois Gouget <fgouget@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

winex11: Set res_class to the program name for Crostini.
6eb05dab · Francois Gouget · Alexandre Julliard · 6bdb2210 · 6eb05dab
Commit 6eb05dab authored Apr 17, 2020 by Francois Gouget Committed by Alexandre Julliard Apr 17, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 3 deletions

window.c dlls/winex11.drv/window.c +1 -3

No files found.
--- a/dlls/winex11.drv/window.c
+++ b/dlls/winex11.drv/window.c
@@ -822,10 +822,8 @@ static void set_initial_wm_hints( Display *display, Window window )
    /* class hints */
    if ((class_hints = XAllocClassHint()))
    {
-        static char wine[] = "Wine";
-
        class_hints->res_name = process_name;
-        class_hints->res_class = wine;
+        class_hints->res_class = process_name;
        XSetClassHint( display, window, class_hints );
        XFree( class_hints );
    }