From e74d3e73b455fe574bfc83ad453f4a8f2f87ecea Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Sun, 1 Jun 2014 16:28:38 +0000 Subject: [PATCH] Relax checking window visibility in wxGLCanvas::SetCurrent(). It is enough for the window to be shown for SetCurrent() to work, it doesn't have to be actually visible on screen, and checking for this using IsShownOnScreen() resulted in false positives. Closes #16193. git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@76644 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- src/common/glcmn.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/common/glcmn.cpp b/src/common/glcmn.cpp index 09d9dd6582..ff9372bd50 100644 --- a/src/common/glcmn.cpp +++ b/src/common/glcmn.cpp @@ -56,7 +56,7 @@ bool wxGLCanvasBase::SetCurrent(const wxGLContext& context) const // although on MSW it works even if the window is still hidden, it doesn't // work in other ports (notably X11-based ones) and documentation mentions // that SetCurrent() can only be called for a shown window, so check for it - wxASSERT_MSG( IsShownOnScreen(), wxT("can't make hidden GL canvas current") ); + wxASSERT_MSG( IsShown(), wxT("can't make hidden GL canvas current") ); return context.SetCurrent(*static_cast(this));