surface: emit unmap before unmapping and releasing the buffers

src/protocols/LayerShell.cpp

@@ -73,8 +73,8 @@ CLayerShellResource::CLayerShellResource(SP<CZwlrLayerSurfaceV1> resource_, SP<C
 
         if (!attachedBuffer && mapped) {
             mapped = false;
-            surface->unmap();
             events.unmap.emit();
+            surface->unmap();
             configured = false;
             return;
         }

src/protocols/XDGShell.cpp

@@ -365,8 +365,8 @@ CXDGSurfaceResource::CXDGSurfaceResource(SP<CXdgSurface> resource_, SP<CXDGWMBas
 
         if (!surface->current.buffer && mapped) {
             mapped = false;
-            surface->unmap();
             events.unmap.emit();
+            surface->unmap();
             return;
         }
 

src/protocols/core/Compositor.cpp

@@ -152,8 +152,8 @@ CWLSurfaceResource::~CWLSurfaceResource() {
 
 void CWLSurfaceResource::destroy() {
     if (mapped) {
-        unmap();
         events.unmap.emit();
+        unmap();
     }
     events.destroy.emit();
     PROTO::compositor->destroyResource(this);

src/protocols/core/Subcompositor.cpp

@@ -87,8 +87,8 @@ CWLSubsurfaceResource::CWLSubsurfaceResource(SP<CWlSubsurface> resource_, SP<CWL
         }
 
         if (!surface->current.buffer && surface->mapped) {
-            surface->unmap();
             surface->events.unmap.emit();
+            surface->unmap();
             return;
         }
     });

src/xwayland/XSurface.cpp

@@ -114,12 +114,11 @@ void CXWaylandSurface::unmap() {
     std::erase(g_pXWayland->pWM->mappedSurfacesStacking, self);
 
     mapped = false;
+    events.unmap.emit();
     surface->unmap();
 
     Debug::log(LOG, "XWayland surface {:x} unmapping", (uintptr_t)this);
 
-    events.unmap.emit();
-
     g_pXWayland->pWM->updateClientList();
 }