From 360e63b8a73836b85b44980ce7e0f77348d43df3 Mon Sep 17 00:00:00 2001
From: emersion <contact@emersion.fr>
Date: Sat, 12 May 2018 19:33:38 +0100
Subject: [PATCH] Don't destroy view child on xdg-popup unmap

---
 include/sway/tree/view.h    | 1 -
 sway/desktop/xdg_shell_v6.c | 8 --------
 sway/desktop/xwayland.c     | 4 ----
 3 files changed, 13 deletions(-)

diff --git a/include/sway/tree/view.h b/include/sway/tree/view.h
index 4ecd8c44d5..8091fe0c73 100644
--- a/include/sway/tree/view.h
+++ b/include/sway/tree/view.h
@@ -153,7 +153,6 @@ struct sway_xdg_popup_v6 {
 	struct wlr_xdg_surface_v6 *wlr_xdg_surface_v6;
 
 	struct wl_listener new_popup;
-	struct wl_listener unmap;
 	struct wl_listener destroy;
 };
 
diff --git a/sway/desktop/xdg_shell_v6.c b/sway/desktop/xdg_shell_v6.c
index 8ecb330da6..e9051b6cc3 100644
--- a/sway/desktop/xdg_shell_v6.c
+++ b/sway/desktop/xdg_shell_v6.c
@@ -20,7 +20,6 @@ static void popup_destroy(struct sway_view_child *child) {
 	}
 	struct sway_xdg_popup_v6 *popup = (struct sway_xdg_popup_v6 *)child;
 	wl_list_remove(&popup->new_popup.link);
-	wl_list_remove(&popup->unmap.link);
 	wl_list_remove(&popup->destroy.link);
 	free(popup);
 }
@@ -39,11 +38,6 @@ static void popup_handle_new_popup(struct wl_listener *listener, void *data) {
 	popup_create(wlr_popup, popup->child.view);
 }
 
-static void popup_handle_unmap(struct wl_listener *listener, void *data) {
-	struct sway_xdg_popup_v6 *popup = wl_container_of(listener, popup, unmap);
-	view_child_destroy(&popup->child);
-}
-
 static void popup_handle_destroy(struct wl_listener *listener, void *data) {
 	struct sway_xdg_popup_v6 *popup = wl_container_of(listener, popup, destroy);
 	view_child_destroy(&popup->child);
@@ -62,8 +56,6 @@ static struct sway_xdg_popup_v6 *popup_create(
 
 	wl_signal_add(&xdg_surface->events.new_popup, &popup->new_popup);
 	popup->new_popup.notify = popup_handle_new_popup;
-	wl_signal_add(&xdg_surface->events.unmap, &popup->unmap);
-	popup->unmap.notify = popup_handle_unmap;
 	wl_signal_add(&xdg_surface->events.destroy, &popup->destroy);
 	popup->destroy.notify = popup_handle_destroy;
 
diff --git a/sway/desktop/xwayland.c b/sway/desktop/xwayland.c
index 8f93576020..1d3c857db5 100644
--- a/sway/desktop/xwayland.c
+++ b/sway/desktop/xwayland.c
@@ -82,10 +82,6 @@ static void unmanaged_handle_unmap(struct wl_listener *listener, void *data) {
 static void unmanaged_handle_destroy(struct wl_listener *listener, void *data) {
 	struct sway_xwayland_unmanaged *surface =
 		wl_container_of(listener, surface, destroy);
-	struct wlr_xwayland_surface *xsurface = surface->wlr_xwayland_surface;
-	if (xsurface->mapped) {
-		unmanaged_handle_unmap(&surface->unmap, xsurface);
-	}
 	wl_list_remove(&surface->map.link);
 	wl_list_remove(&surface->unmap.link);
 	wl_list_remove(&surface->destroy.link);