compositor: unmap subsurfaces too on unmap

Similar to 49e9be62ae.

types/wlr_compositor.c

@@ -754,6 +754,14 @@ void wlr_surface_unmap(struct wlr_surface *surface) {
 	if (surface->role != NULL && surface->role->unmap != NULL) {
 		surface->role->unmap(surface);
 	}
+
+	struct wlr_subsurface *subsurface;
+	wl_list_for_each(subsurface, &surface->current.subsurfaces_below, current.link) {
+		wlr_surface_unmap(subsurface->surface);
+	}
+	wl_list_for_each(subsurface, &surface->current.subsurfaces_above, current.link) {
+		wlr_surface_unmap(subsurface->surface);
+	}
 }
 
 bool wlr_surface_set_role(struct wlr_surface *surface,

types/wlr_subcompositor.c

@@ -220,22 +220,6 @@ static void subsurface_role_commit(struct wlr_surface *surface) {
 	subsurface_consider_map(subsurface);
 }
 
-static void subsurface_role_unmap(struct wlr_surface *surface) {
-	struct wlr_subsurface *subsurface = wlr_subsurface_try_from_wlr_surface(surface);
-	assert(subsurface != NULL);
-
-	// Unmap all children
-	struct wlr_subsurface *child;
-	wl_list_for_each(child, &subsurface->surface->current.subsurfaces_below,
-			current.link) {
-		wlr_surface_unmap(child->surface);
-	}
-	wl_list_for_each(child, &subsurface->surface->current.subsurfaces_above,
-			current.link) {
-		wlr_surface_unmap(child->surface);
-	}
-}
-
 static void subsurface_role_destroy(struct wlr_surface *surface) {
 	struct wlr_subsurface *subsurface = wlr_subsurface_try_from_wlr_surface(surface);
 	assert(subsurface != NULL);
@@ -259,7 +243,6 @@ static void subsurface_role_destroy(struct wlr_surface *surface) {
 const struct wlr_surface_role subsurface_role = {
 	.name = "wl_subsurface",
 	.commit = subsurface_role_commit,
-	.unmap = subsurface_role_unmap,
 	.destroy = subsurface_role_destroy,
 };