vcl: Don't create new contexts for Virtual Devices on Windows

Change-Id: I561a8142f986aca89e796ce2c4a0902fae41f9e6

include/vcl/opengl/OpenGLContext.hxx

@@ -238,8 +238,6 @@ public:
         return mbInitialized;
     }
 
-    void resetToReInitialize();
-
     bool supportMultiSampling() const;
 
     static SystemWindowData generateWinData(vcl::Window* pParent, bool bRequestLegacyContext);

vcl/inc/opengl/win/gdiimpl.hxx

@@ -30,7 +30,6 @@ public:
 protected:
     virtual OpenGLContext* CreateWinContext() SAL_OVERRIDE;
     virtual bool UseContext( OpenGLContext* pContext ) SAL_OVERRIDE;
-    virtual OpenGLContext* CreatePixmapContext() SAL_OVERRIDE;
 
 public:
     virtual void copyBits( const SalTwoRect& rPosAry, SalGraphics* pSrcGraphics ) SAL_OVERRIDE;

vcl/inc/opengl/x11/gdiimpl.hxx

@@ -27,7 +27,6 @@ public:
 
 protected:
     virtual OpenGLContext* CreateWinContext() SAL_OVERRIDE;
-    virtual OpenGLContext* CreatePixmapContext() SAL_OVERRIDE;
     virtual bool UseContext( OpenGLContext* pContext ) SAL_OVERRIDE;
 
 public:

vcl/inc/openglgdiimpl.hxx

@@ -110,12 +110,12 @@ protected:
     bool AcquireContext();
     bool ReleaseContext();
 
+    // retrieve the default context for offscreen rendering
+    virtual OpenGLContext* GetDefaultContext();
+
     // create a new context for window rendering
     virtual OpenGLContext* CreateWinContext() = 0;
 
-    // create a new context for offscreen rendering
-    virtual OpenGLContext* CreatePixmapContext() = 0;
-
     // check whether the given context can be used by this instance
     virtual bool UseContext( OpenGLContext* pContext ) = 0;
 

vcl/opengl/gdiimpl.cxx

@@ -61,6 +61,11 @@ OpenGLContext* OpenGLSalGraphicsImpl::GetOpenGLContext()
     return mpContext;
 }
 
+OpenGLContext* OpenGLSalGraphicsImpl::GetDefaultContext()
+{
+    return ImplGetDefaultWindow()->GetGraphics()->GetOpenGLContext();
+}
+
 bool OpenGLSalGraphicsImpl::AcquireContext( )
 {
     ImplSVData* pSVData = ImplGetSVData();
@@ -81,7 +86,7 @@ bool OpenGLSalGraphicsImpl::AcquireContext( )
     if( pContext )
         pContext->AddRef();
     else
-        pContext = mbOffscreen ? CreatePixmapContext() : CreateWinContext();
+        pContext = mbOffscreen ? GetDefaultContext() : CreateWinContext();
 
     mpContext = pContext;
     return (mpContext != NULL);
@@ -112,15 +117,6 @@ void OpenGLSalGraphicsImpl::Init()
         maOffscreenTex.GetHeight() != GetHeight() )
     {
         maOffscreenTex = OpenGLTexture();
-#if defined(WNT)
-        // URGH ... VirtualDevice may have destroyed the underlying resource
-        // our context is associated with - FIXME: can we do better here ?
-        if (mpContext)
-        {
-            mpContext->resetToReInitialize();
-            ReleaseContext();
-        }
-#endif
     }
 }
 

vcl/opengl/win/gdiimpl.cxx

@@ -38,16 +38,9 @@ bool WinOpenGLSalGraphicsImpl::UseContext( OpenGLContext* pContext )
 {
     if( !pContext || !pContext->isInitialized() )
         return false;
+    if( IsOffscreen() )
+        return true;
     return ( pContext->getOpenGLWindow().hWnd == mrParent.mhWnd );
 }
 
-OpenGLContext* WinOpenGLSalGraphicsImpl::CreatePixmapContext()
-{
-    OpenGLContext* pContext = new OpenGLContext();
-    pContext->requestVirtualDevice();
-    pContext->requestSingleBufferedRendering();
-    pContext->init( mrParent.mhLocalDC, mrParent.mhWnd );
-    return pContext;
-}
-
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */

vcl/opengl/x11/gdiimpl.cxx

@@ -56,16 +56,6 @@ OpenGLContext* X11OpenGLSalGraphicsImpl::CreateWinContext()
     return pContext;
 }
 
-OpenGLContext* X11OpenGLSalGraphicsImpl::CreatePixmapContext()
-{
-    X11OpenGLSalVirtualDevice* pVDev = dynamic_cast<X11OpenGLSalVirtualDevice*>(mrParent.m_pVDev);
-
-    if( pVDev == NULL )
-        return NULL;
-
-    return ImplGetDefaultWindow()->GetGraphics()->GetOpenGLContext();
-}
-
 bool X11OpenGLSalGraphicsImpl::UseContext( OpenGLContext* pContext )
 {
     X11WindowProvider *pProvider = dynamic_cast<X11WindowProvider*>(mrParent.m_pFrame);

vcl/source/opengl/OpenGLContext.cxx

@@ -633,17 +633,6 @@ bool OpenGLContext::init(SystemChildWindow* pChildWindow)
     return ImplInit();
 }
 
-#if defined( WNT )
-// FIXME share resetToReInitialize() across platforms...
-void OpenGLContext::resetToReInitialize()
-{
-    if( !mbInitialized )
-        return;
-    resetCurrent();
-    mbInitialized = false;
-}
-#endif
-
 #if defined( UNX ) && !defined MACOSX && !defined IOS && !defined ANDROID
 bool OpenGLContext::init(Display* dpy, Window win, int screen)
 {