From: Michael Stapelberg <michael+x200@stapelberg.de>
Date: Thu, 12 Mar 2009 15:44:44 +0000 (+0100)
Subject: Bugfix: Completely ignore legacy hints as soon as the client uses _NET_WM_NAME
X-Git-Tag: 3.a~25
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=755709c86d91c6862738ba5ed3836ac9f6a89637;p=i3%2Fi3

Bugfix: Completely ignore legacy hints as soon as the client uses _NET_WM_NAME
---

diff --git a/include/data.h b/include/data.h
index 5c4bfb0a..21ea722e 100644
--- a/include/data.h
+++ b/include/data.h
@@ -228,6 +228,9 @@ struct Client {
            to X as 8-bit string and therefore will not be rendered correctly. This behaviour is
            to support legacy applications which do not set _NET_WM_NAME */
         int name_len;
+        /* This will be set to true as soon as the first _NET_WM_NAME comes in. If set to true,
+           legacy window names are ignored. */
+        bool uses_net_wm_name;
 
         /* fullscreen is pretty obvious */
         bool fullscreen;
diff --git a/src/handlers.c b/src/handlers.c
index 8ab8ac4d..8278a89a 100644
--- a/src/handlers.c
+++ b/src/handlers.c
@@ -616,6 +616,7 @@ int handle_windowname_change(void *data, xcb_connection_t *conn, uint8_t state,
         char *old_name = client->name;
         client->name = ucs2_name;
         client->name_len = new_len;
+        client->uses_net_wm_name = true;
 
         if (old_name != NULL)
                 free(old_name);
@@ -646,7 +647,7 @@ int handle_windowname_change(void *data, xcb_connection_t *conn, uint8_t state,
 int handle_windowname_change_legacy(void *data, xcb_connection_t *conn, uint8_t state,
                                 xcb_window_t window, xcb_atom_t atom, xcb_get_property_reply_t *prop) {
         LOG("window's name changed (legacy).\n");
-        if (prop == NULL) {
+        if (prop == NULL || xcb_get_property_value_length(prop) == 0) {
                 LOG("prop == NULL\n");
                 return 1;
         }
@@ -654,6 +655,11 @@ int handle_windowname_change_legacy(void *data, xcb_connection_t *conn, uint8_t
         if (client == NULL)
                 return 1;
 
+        if (client->uses_net_wm_name) {
+                LOG("This client is capable of _NET_WM_NAME, ignoring legacy name\n");
+                return 1;
+        }
+
         /* Save the old pointer to make the update atomic */
         char *new_name;
         int new_len;