From: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Date: Tue, 29 Apr 2014 14:54:02 +0000 (+0300)
Subject: shell: Fix memory leaks caused by the window close animation
X-Git-Tag: upstream/0.1.8~252
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=15f9a26bb199c3567098b1b29815cae609010957;p=profile%2Fivi%2Fweston-ivi-shell.git

shell: Fix memory leaks caused by the window close animation

In order to do the window close animation, a reference for a destroyed
surface is kept. However, the reference count was also increased for
unmapped surfaces, in which case the animation wouldn't run. Since the
reference count was decremented in the animation done function, it would
never be decreased for unmapped surfaces, causing them to not be
released.

The close animation also changed how shell surfaces are released. The
destroy function for its resource was changed to not deallocate the
surface, and instead keep it around until the animation finishes and
the weston surface is destroyed. The destruction should happen in the
destroy listener for the weston surface, but it wouldn't destroy the
shell surface in the case the resource was still valid, assuming that
it would be freed in the resource destroy function.
---

diff --git a/desktop-shell/shell.c b/desktop-shell/shell.c
index bc4a258..6fc797b 100644
--- a/desktop-shell/shell.c
+++ b/desktop-shell/shell.c
@@ -3081,8 +3081,8 @@ shell_handle_surface_destroy(struct wl_listener *listener, void *data)
 
 	if (shsurf->resource)
 		wl_resource_destroy(shsurf->resource);
-	else
-		destroy_shell_surface(shsurf);
+
+	destroy_shell_surface(shsurf);
 }
 
 static void
@@ -3100,15 +3100,17 @@ handle_resource_destroy(struct wl_listener *listener, void *data)
 		container_of(listener, struct shell_surface,
 			     resource_destroy_listener);
 
+	if (!weston_surface_is_mapped(shsurf->surface))
+		return;
+
 	shsurf->surface->ref_count++;
 
 	pixman_region32_fini(&shsurf->surface->pending.input);
 	pixman_region32_init(&shsurf->surface->pending.input);
 	pixman_region32_fini(&shsurf->surface->input);
 	pixman_region32_init(&shsurf->surface->input);
-	if (weston_surface_is_mapped(shsurf->surface))
-		weston_fade_run(shsurf->view, 1.0, 0.0, 300.0,
-				fade_out_done, shsurf);
+	weston_fade_run(shsurf->view, 1.0, 0.0, 300.0,
+			fade_out_done, shsurf);
 }
 
 static void