commit 62942ad3a68a775786aae92a10438ccdf2c73fa9
author Giulio Camuffo <giuliocamuffo@gmail.com> Wed Sep 11 18:20:47 2013 +0200
committer Kristian Høgsberg <krh@bitplanet.net> Wed Sep 11 13:50:54 2013 -0700
tree df452cdf1ed852c0d8f83a35c66ba21cf49c8fc9
parent ca43f0942e23356a0a0f3e08e4d20a8a31b4d629

xwm: set the shell_surface's title

add a new function pointer to the weston_shell_interface struct that
shells will set accordingly.

src/compositor.h

diff --git a/src/compositor.h b/src/compositor.h
index 50e96ff..a523c83 100644
--- a/src/compositor.h
+++ b/src/compositor.h
@@ -107,6 +107,8 @@
 	int (*move)(struct shell_surface *shsurf, struct weston_seat *ws);
 	int (*resize)(struct shell_surface *shsurf,
 		      struct weston_seat *ws, uint32_t edges);
+	void (*set_title)(struct shell_surface *shsurf,
+	                  const char *title);
 
 };
 

src/shell.c

diff --git a/src/shell.c b/src/shell.c
index 89153e3..c1e0109 100644
--- a/src/shell.c
+++ b/src/shell.c
@@ -1592,13 +1592,19 @@
 }
 
 static void
+set_title(struct shell_surface *shsurf, const char *title)
+{
+	free(shsurf->title);
+	shsurf->title = strdup(title);
+}
+
+static void
 shell_surface_set_title(struct wl_client *client,
 			struct wl_resource *resource, const char *title)
 {
 	struct shell_surface *shsurf = wl_resource_get_user_data(resource);
 
-	free(shsurf->title);
-	shsurf->title = strdup(title);
+	set_title(shsurf, title);
 }
 
 static void
@@ -4580,6 +4586,7 @@
 	ec->shell_interface.set_xwayland = set_xwayland;
 	ec->shell_interface.move = surface_move;
 	ec->shell_interface.resize = surface_resize;
+	ec->shell_interface.set_title = set_title;
 
 	wl_list_init(&shell->input_panel.surfaces);
 

src/xwayland/window-manager.c

diff --git a/src/xwayland/window-manager.c b/src/xwayland/window-manager.c
index f775734..b4f64d3 100644
--- a/src/xwayland/window-manager.c
+++ b/src/xwayland/window-manager.c
@@ -363,6 +363,8 @@
 weston_wm_window_read_properties(struct weston_wm_window *window)
 {
 	struct weston_wm *wm = window->wm;
+	struct weston_shell_interface *shell_interface =
+		&wm->server->compositor->shell_interface;
 
 #define F(field) offsetof(struct weston_wm_window, field)
 	const struct {
@@ -468,6 +470,9 @@
 		}
 		free(reply);
 	}
+
+	if (window->shsurf && window->name)
+		shell_interface->set_title(window->shsurf, window->name);
 }
 
 static void
@@ -1875,6 +1880,9 @@
 
 	wm_log("surface for xid %d destroyed\n", window->id);
 
+	/* This should have been freed by the shell.
+       Don't try to use it later. */
+	window->shsurf = NULL;
 	window->surface = NULL;
 }
 
@@ -2029,6 +2037,9 @@
 						      window->surface,
 						      &shell_client);
 
+	if (window->name)
+		shell_interface->set_title(window->shsurf, window->name);
+
 	if (window->fullscreen) {
 		window->saved_width = window->width;
 		window->saved_height = window->height;