From 8801357332b7a239e0d1b84cae87334d3fbd80db Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Philipp=20Br=C3=BCschweiler?= <blei42@gmail.com>
Date: Mon, 6 Aug 2012 13:44:42 +0200
Subject: [PATCH] shell: don't abuse link_layer for keeping track of input
 panel surfaces

Input panel surfaces were kept in a list by using layer_link of
weston_surface. This was pretty hacky and resulted in the bug that
an input panel surface was not removed from the list if it was unmapped
at the time of destruction.

This patch wraps the surface in a new input_panel_surface struct and
properly handles destruction with a signal listener.
---
 src/shell.c | 57 +++++++++++++++++++++++++++++++++++++++++------------
 1 file changed, 44 insertions(+), 13 deletions(-)

diff --git a/src/shell.c b/src/shell.c
index 6c810ff9..35b19a1a 100644
--- a/src/shell.c
+++ b/src/shell.c
@@ -63,6 +63,12 @@ struct workspace {
 	struct wl_listener seat_destroyed_listener;
 };
 
+struct input_panel_surface {
+	struct wl_list link;
+	struct weston_surface *surface;
+	struct wl_listener listener;
+};
+
 struct desktop_shell {
 	struct weston_compositor *compositor;
 
@@ -2507,17 +2513,17 @@ show_input_panels(struct wl_listener *listener, void *data)
 {
 	struct desktop_shell *shell =
 		container_of(listener, struct desktop_shell, show_input_panel_listener);
-	struct weston_surface *surface, *next;
+	struct input_panel_surface *surface, *next;
+	struct weston_surface *ws;
 
 	wl_list_for_each_safe(surface, next,
-			      &shell->input_panel.surfaces, layer_link) {
-		wl_list_remove(&surface->layer_link);
+			      &shell->input_panel.surfaces, link) {
+		ws = surface->surface;
 		wl_list_insert(&shell->panel_layer.surface_list,
-			       &surface->layer_link);
-		weston_surface_assign_output(surface);
-		weston_surface_damage(surface);
-		weston_slide_run(surface,
-				 surface->geometry.height, 0, NULL, NULL);
+			       &ws->layer_link);
+		weston_surface_assign_output(ws);
+		weston_surface_damage(ws);
+		weston_slide_run(ws, ws->geometry.height, 0, NULL, NULL);
 	}
 }
 
@@ -2533,11 +2539,8 @@ hide_input_panels(struct wl_listener *listener, void *data)
 
 	wl_list_for_each_safe(surface, next,
 			      &shell->panel_layer.surface_list, layer_link)
-		if (surface->configure == input_panel_configure) {
+		if (surface->configure == input_panel_configure)
 			weston_surface_unmap(surface);
-			wl_list_insert(&shell->input_panel.surfaces,
-				       &surface->layer_link);
-		}
 }
 
 static void
@@ -2897,6 +2900,19 @@ input_panel_configure(struct weston_surface *surface, int32_t sx, int32_t sy)
 				 surface->buffer->height);
 }
 
+static void
+destroy_input_panel_surface(struct wl_listener *listener,
+			    void *data)
+{
+	struct input_panel_surface *input_panel_surface =
+		container_of(listener, struct input_panel_surface, listener);
+
+	wl_list_remove(&listener->link);
+	wl_list_remove(&input_panel_surface->link);
+
+	free(input_panel_surface);
+}
+
 static void
 input_panel_set_surface(struct wl_client *client,
 			struct wl_resource *resource,
@@ -2906,11 +2922,26 @@ input_panel_set_surface(struct wl_client *client,
 	struct desktop_shell *shell = resource->data;
 	struct weston_surface *surface = surface_resource->data;
 	struct weston_output *output = output_resource->data;
+	struct input_panel_surface *input_panel_surface;
 
 	surface->configure = input_panel_configure;
 	surface->private = shell;
 	surface->output = output;
-	wl_list_insert(shell->input_panel.surfaces.prev, &surface->layer_link);
+
+	input_panel_surface = malloc(sizeof *input_panel_surface);
+	if (!input_panel_surface) {
+		wl_resource_post_no_memory(resource);
+		return;
+	}
+
+	input_panel_surface->surface = surface;
+	input_panel_surface->listener.notify = destroy_input_panel_surface;
+
+	wl_signal_add(&surface_resource->destroy_signal,
+		      &input_panel_surface->listener);
+
+	wl_list_insert(&shell->input_panel.surfaces,
+		       &input_panel_surface->link);
 }
 
 static const struct input_panel_interface input_panel_implementation = {