Add a mechanism to delay unmapping of surfaces

When a client attachs a NULL buffer on a surface we may want to hang on
to the old buffer for a bit, e.g. to do a hide animation.
Add the QWaylandUnmapLock class to keep a surface mapped after an attach(0)
request.

Change-Id: I5d5be0b36410a76a58d2b68c1b2ac5e3b875a801
Reviewed-by: Robin Burchell <robin.burchell@viroteck.net>

5 files changed, 50 insertions, 17 deletions

diff --git a/src/compositor/compositor_api/qwaylandbufferref.h b/src/compositor/compositor_api/qwaylandbufferref.h
index 213474fab..103a1b388 100644
--- a/src/compositor/compositor_api/qwaylandbufferref.h
+++ b/src/compositor/compositor_api/qwaylandbufferref.h
@@ -65,7 +65,8 @@ public:
 #ifdef QT_COMPOSITOR_WAYLAND_GL
     /**
      * There must be a GL context bound when calling this function.
-     * It is responsibility of the caller to call destroyTexture() later.
+     * The texture will be automatically destroyed when the last QWaylandBufferRef
+     * referring to the same underlying buffer will be destroyed or reset.
      */
     GLuint createTexture();
     void destroyTexture();
diff --git a/src/compositor/compositor_api/qwaylandquicksurface.cpp b/src/compositor/compositor_api/qwaylandquicksurface.cpp
index ef78c8849..c8edcb6bc 100644
--- a/src/compositor/compositor_api/qwaylandquicksurface.cpp
+++ b/src/compositor/compositor_api/qwaylandquicksurface.cpp
@@ -73,19 +73,12 @@ public:
 
     void createTexture()
     {
-        if (bufferRef)
-            bufferRef.destroyTexture();
         bufferRef = nextBuffer;
+        delete texture;
+        texture = 0;
 
         QQuickWindow *window = static_cast<QQuickWindow *>(surface->mainOutput()->window());
-
-        // If the next buffer is NULL do not delete the current texture. If the client called
-        // attach(0) the surface is going to be unmapped anyway, if instead the client attached
-        // a valid buffer but died before we got here we want to keep the old buffer around
-        // in case some destroy animation is run.
-        if (bufferRef) {
-            delete texture;
-
+        if (nextBuffer) {
             if (bufferRef.isShm()) {
                 texture = window->createTextureFromImage(bufferRef.image());
             } else {
@@ -101,6 +94,12 @@ public:
         update = false;
     }
 
+    void unmap() Q_DECL_OVERRIDE
+    {
+        nextBuffer = QWaylandBufferRef();
+        update = true;
+    }
+
     void invalidateTexture()
     {
         if (bufferRef)
@@ -141,9 +140,6 @@ public:
 
     void surface_commit(Resource *resource) Q_DECL_OVERRIDE
     {
-        if (m_pending.newlyAttached) {
-            buffer->update = true;
-        }
         QWaylandSurfacePrivate::surface_commit(resource);
 
         Q_FOREACH (QtWayland::Output *output, outputs())
diff --git a/src/compositor/compositor_api/qwaylandsurface.cpp b/src/compositor/compositor_api/qwaylandsurface.cpp
index f71504c65..d1bcb6efb 100644
--- a/src/compositor/compositor_api/qwaylandsurface.cpp
+++ b/src/compositor/compositor_api/qwaylandsurface.cpp
@@ -511,4 +511,30 @@ void QWaylandSurfacePrivate::setType(QWaylandSurface::WindowType type)
     }
 }
 
+class QWaylandUnmapLockPrivate
+{
+public:
+    QWaylandSurface *surface;
+};
+
+/*!
+    Constructs a QWaylandUnmapLock object.
+
+    The lock will act on the \a surface parameter, and will prevent the surface to
+    be unmapped, retaining the last valid buffer when the client attachs a NULL buffer.
+    The lock will be automatically released when deleted.
+*/
+QWaylandUnmapLock::QWaylandUnmapLock(QWaylandSurface *surface)
+                 : d(new QWaylandUnmapLockPrivate)
+{
+    d->surface = surface;
+    surface->handle()->addUnmapLock(this);
+}
+
+QWaylandUnmapLock::~QWaylandUnmapLock()
+{
+    d->surface->handle()->removeUnmapLock(this);
+    delete d;
+}
+
 QT_END_NAMESPACE
diff --git a/src/compositor/compositor_api/qwaylandsurface.h b/src/compositor/compositor_api/qwaylandsurface.h
index 653d74c1b..db1e1fb44 100644
--- a/src/compositor/compositor_api/qwaylandsurface.h
+++ b/src/compositor/compositor_api/qwaylandsurface.h
@@ -77,6 +77,7 @@ public:
 
 protected:
     virtual void attach(const QWaylandBufferRef &ref) = 0;
+    virtual void unmap() = 0;
 
     friend class QtWayland::Surface;
 };
@@ -261,6 +262,17 @@ Q_SIGNALS:
     friend class QtWayland::Surface;
 };
 
+class QWaylandUnmapLockPrivate;
+class Q_COMPOSITOR_EXPORT QWaylandUnmapLock
+{
+public:
+    QWaylandUnmapLock(QWaylandSurface *surface);
+    ~QWaylandUnmapLock();
+
+private:
+    QWaylandUnmapLockPrivate *const d;
+};
+
 QT_END_NAMESPACE
 
 #endif // QWAYLANDSURFACE_H
diff --git a/src/compositor/compositor_api/qwaylandsurfaceitem.cpp b/src/compositor/compositor_api/qwaylandsurfaceitem.cpp
index 93cfaf008..0c48df382 100644
--- a/src/compositor/compositor_api/qwaylandsurfaceitem.cpp
+++ b/src/compositor/compositor_api/qwaylandsurfaceitem.cpp
@@ -364,9 +364,7 @@ void QWaylandSurfaceItem::updateTexture(bool changed)
     if (!m_provider)
         m_provider = new QWaylandSurfaceTextureProvider();
 
-    bool mapped = surface() && surface()->isMapped();
-    if (mapped)
-        m_provider->t = static_cast<QWaylandQuickSurface *>(surface())->texture();
+    m_provider->t = static_cast<QWaylandQuickSurface *>(surface())->texture();
     m_provider->smooth = smooth();
     if (m_newTexture || changed)
         emit m_provider->textureChanged();