Push the "lazy doneCurrent" logic down from QtOpenVG into QEgl

QtOpenVG was doing a lot of housekeeping to avoid having to
switch EGL contexts if the same surface was used over and over.
This housekeeping really belongs in the QEgl layer so that
QtOpenGL could potentially use it as well.

This change also adds some overrides for makeCurrent(),
swapBuffers(), and destroySurface() that take an EGLSurface
directly.  This is the first step in separating EGL surface
management from context management.

Reviewed-by: Sarah Smith

1 files changed, 27 insertions, 96 deletions

diff --git a/src/openvg/qwindowsurface_vgegl.cpp b/src/openvg/qwindowsurface_vgegl.cpp
index 06759d4f8d..cb3f7fc925 100644
--- a/src/openvg/qwindowsurface_vgegl.cpp
+++ b/src/openvg/qwindowsurface_vgegl.cpp
@@ -125,7 +125,6 @@ public:
     int refCount;
     QVGPaintEngine *engine;
     EGLSurface surface;
-    EGLSurface lastSurface;
 };
 
 QVGSharedContext::QVGSharedContext()
@@ -133,7 +132,6 @@ QVGSharedContext::QVGSharedContext()
     , refCount(0)
     , engine(0)
     , surface(EGL_NO_SURFACE)
-    , lastSurface(EGL_NO_SURFACE)
 {
 }
 
@@ -144,12 +142,12 @@ QVGSharedContext::~QVGSharedContext()
     ++refCount;
 
     if (context)
-        qt_vg_make_current(context, qt_vg_shared_surface());
+        context->makeCurrent(qt_vg_shared_surface());
     delete engine;
     if (context)
-        qt_vg_done_current(context, true);
-    if (surface != EGL_NO_SURFACE)
-        qt_vg_destroy_surface(context, surface);
+        context->doneCurrent();
+    if (context && surface != EGL_NO_SURFACE)
+        context->destroySurface(surface);
     delete context;
 }
 
@@ -265,12 +263,12 @@ void qt_vg_destroy_context(QEglContext *context)
         // This is not the shared context.  Shouldn't happen!
         delete context;
     } else if (--(shared->refCount) <= 0) {
-        qt_vg_make_current(shared->context, qt_vg_shared_surface());
+        shared->context->makeCurrent(qt_vg_shared_surface());
         delete shared->engine;
         shared->engine = 0;
-        qt_vg_done_current(shared->context, true);
+        shared->context->doneCurrent();
         if (shared->surface != EGL_NO_SURFACE) {
-            qt_vg_destroy_surface(shared->context, shared->surface);
+            eglDestroySurface(shared->context->display(), shared->surface);
             shared->surface = EGL_NO_SURFACE;
         }
         delete shared->context;
@@ -303,50 +301,6 @@ EGLSurface qt_vg_shared_surface(void)
     return shared->surface;
 }
 
-void qt_vg_make_current(QEglContext *context, EGLSurface surface)
-{
-    // Bail out if the context and surface are already current.
-    if (context->isCurrent() && context->surface() == surface)
-        return;
-
-    // Are we setting the surface to the same as the last elided doneCurrent()?
-    QVGSharedContext *shared = sharedContext();
-    if (context->isCurrent() && shared->lastSurface == surface) {
-        shared->lastSurface = EGL_NO_SURFACE;
-        context->setSurface(surface);
-        return;
-    }
-
-    // Switch to the new context and surface.
-    shared->lastSurface = EGL_NO_SURFACE;
-    context->setSurface(surface);
-    context->makeCurrent();
-}
-
-void qt_vg_done_current(QEglContext *context, bool force)
-{
-    QVGSharedContext *shared = sharedContext();
-    if (force) {
-        context->doneCurrent();
-        shared->lastSurface = EGL_NO_SURFACE;
-    } else {
-        // Keep the context current for now just in case we immediately
-        // reuse the same surface for the next frame.
-        shared->lastSurface = context->surface();
-    }
-}
-
-void qt_vg_destroy_surface(QEglContext *context, EGLSurface surface)
-{
-    QVGSharedContext *shared = sharedContext();
-    if (shared->lastSurface == surface) {
-        shared->lastSurface = EGL_NO_SURFACE;
-        context->doneCurrent();
-    }
-    context->setSurface(surface);
-    context->destroySurface();
-}
-
 #else
 
 QEglContext *qt_vg_create_context(QPaintDevice *device)
@@ -364,24 +318,6 @@ EGLSurface qt_vg_shared_surface(void)
     return EGL_NO_SURFACE;
 }
 
-void qt_vg_make_current(QEglContext *context, EGLSurface surface)
-{
-    context->setSurface(surface);
-    context->makeCurrent();
-}
-
-void qt_vg_done_current(QEglContext *context, bool force)
-{
-    Q_UNUSED(force);
-    context->doneCurrent();
-}
-
-void qt_vg_destroy_surface(QEglContext *context, EGLSurface surface)
-{
-    context->setSurface(surface);
-    context->destroySurface();
-}
-
 #endif
 
 QVGEGLWindowSurfacePrivate::QVGEGLWindowSurfacePrivate(QWindowSurface *win)
@@ -470,13 +406,13 @@ QVGEGLWindowSurfaceVGImage::~QVGEGLWindowSurfaceVGImage()
             // We need a current context to be able to destroy the image.
             // We use the shared surface because the native window handle
             // associated with "windowSurface" may have been destroyed already.
-            qt_vg_make_current(context, qt_vg_shared_surface());
-            qt_vg_destroy_surface(context, backBufferSurface);
+            context->makeCurrent(qt_vg_shared_surface());
+            context->destroySurface(backBufferSurface);
             vgDestroyImage(backBuffer);
-            qt_vg_done_current(context, true);
+            context->doneCurrent();
         }
         if (windowSurface != EGL_NO_SURFACE)
-            qt_vg_destroy_surface(context, windowSurface);
+            context->destroySurface(windowSurface);
         qt_vg_destroy_context(context);
     }
 }
@@ -489,7 +425,7 @@ QEglContext *QVGEGLWindowSurfaceVGImage::ensureContext(QWidget *widget)
         // the back buffer.  Keep the same context and paint engine.
         size = newSize;
         if (isPaintingActive)
-            qt_vg_done_current(context, true);
+            context->doneCurrent();
         isPaintingActive = false;
         recreateBackBuffer = true;
     }
@@ -512,7 +448,7 @@ void QVGEGLWindowSurfaceVGImage::beginPaint(QWidget *widget)
             // Create a VGImage object to act as the back buffer
             // for this window.  We have to create the VGImage with a
             // current context, so activate the main surface for the window.
-            qt_vg_make_current(context, mainSurface());
+            context->makeCurrent(mainSurface());
             recreateBackBuffer = false;
             if (backBufferSurface != EGL_NO_SURFACE) {
                 eglDestroySurface(context->display(), backBufferSurface);
@@ -538,9 +474,9 @@ void QVGEGLWindowSurfaceVGImage::beginPaint(QWidget *widget)
             }
         }
         if (backBufferSurface != EGL_NO_SURFACE)
-            qt_vg_make_current(context, backBufferSurface);
+            context->makeCurrent(backBufferSurface);
         else
-            qt_vg_make_current(context, mainSurface());
+            context->makeCurrent(mainSurface());
         isPaintingActive = true;
     }
 }
@@ -555,9 +491,8 @@ void QVGEGLWindowSurfaceVGImage::endPaint
         if (backBufferSurface != EGL_NO_SURFACE) {
             if (isPaintingActive)
                 vgFlush();
-            qt_vg_done_current(context);
+            context->lazyDoneCurrent();
         }
-        context->setSurface(EGL_NO_SURFACE);
         isPaintingActive = false;
     }
 }
@@ -592,7 +527,7 @@ void QVGEGLWindowSurfaceQImage::endPaint
         if (backBufferSurface != EGL_NO_SURFACE) {
             if (isPaintingActive)
                 vgFlush();
-            qt_vg_make_current(context, mainSurface());
+            context->makeCurrent(mainSurface());
             QRegion rgn = region.intersected
                 (QRect(0, 0, image->width(), image->height()));
             if (rgn.numRects() == 1) {
@@ -602,9 +537,8 @@ void QVGEGLWindowSurfaceQImage::endPaint
                 for (int index = 0; index < rects.size(); ++index)
                     copySubImage(image, backBuffer, rects[index]);
             }
-            qt_vg_done_current(context);
+            context->lazyDoneCurrent();
         }
-        context->setSurface(EGL_NO_SURFACE);
         isPaintingActive = false;
     }
 }
@@ -625,7 +559,7 @@ QVGEGLWindowSurfaceDirect::~QVGEGLWindowSurfaceDirect()
     destroyPaintEngine();
     if (context) {
         if (windowSurface != EGL_NO_SURFACE)
-            qt_vg_destroy_surface(context, windowSurface);
+            context->destroySurface(windowSurface);
         qt_vg_destroy_context(context);
     }
 }
@@ -642,9 +576,8 @@ QEglContext *QVGEGLWindowSurfaceDirect::ensureContext(QWidget *widget)
         // We can keep the same context and paint engine.
         size = newSize;
         if (isPaintingActive)
-            qt_vg_done_current(context, true);
-        context->setSurface(windowSurface);
-        context->destroySurface();
+            context->doneCurrent();
+        context->destroySurface(windowSurface);
 #if defined(EGL_VG_ALPHA_FORMAT_PRE_BIT)
         if (isPremultipliedContext(context)) {
             surfaceProps.setValue
@@ -667,8 +600,7 @@ QEglContext *QVGEGLWindowSurfaceDirect::ensureContext(QWidget *widget)
         // in the new context.
         qt_vg_destroy_paint_engine(engine);
         engine = 0;
-        context->setSurface(windowSurface);
-        context->destroySurface();
+        context->destroySurface(windowSurface);
         qt_vg_destroy_context(context);
         context = 0;
         windowSurface = EGL_NO_SURFACE;
@@ -730,7 +662,7 @@ void QVGEGLWindowSurfaceDirect::beginPaint(QWidget *widget)
 {
     QEglContext *context = ensureContext(widget);
     if (context) {
-        qt_vg_make_current(context, windowSurface);
+        context->makeCurrent(windowSurface);
         isPaintingActive = true;
     }
 }
@@ -744,14 +676,13 @@ void QVGEGLWindowSurfaceDirect::endPaint
     if (context) {
         if (needToSwap) {
             if (!isPaintingActive)
-                qt_vg_make_current(context, windowSurface);
-            context->swapBuffers();
-            qt_vg_done_current(context);
+                context->makeCurrent(windowSurface);
+            context->swapBuffers(windowSurface);
+            context->lazyDoneCurrent();
         } else if (isPaintingActive) {
             vgFlush();
-            qt_vg_done_current(context);
+            context->lazyDoneCurrent();
         }
-        context->setSurface(EGL_NO_SURFACE);
         isPaintingActive = false;
     }
 }