From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Sat, 16 Aug 2008 10:53:02 +0000 (+0000)
Subject: fix spurious assert failure in DestroyGripper() which happened if the dialog hadn... 
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/f78c70f34bc54ada79763b7e1eb1a863a61f542a

fix spurious assert failure in DestroyGripper() which happened if the dialog hadn't been shown at all (#9824)

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@55102 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/src/msw/dialog.cpp b/src/msw/dialog.cpp
index 1b66274cc1..f65ece0b5d 100644
--- a/src/msw/dialog.cpp
+++ b/src/msw/dialog.cpp
@@ -391,8 +391,11 @@ void wxDialog::DestroyGripper()
     {
         // we used to have trouble with gripper appearing on top (and hence
         // overdrawing) the other, real, dialog children -- check that this
-        // isn't the case automatically
-        wxASSERT_MSG( ::GetNextWindow((HWND)m_hGripper, GW_HWNDNEXT) == 0,
+        // isn't the case automatically (but notice that this could be false if
+        // we're not shown at all as in this case ResizeGripper() might not
+        // have been called yet)
+        wxASSERT_MSG( !IsShown() ||
+                      ::GetNextWindow((HWND)m_hGripper, GW_HWNDNEXT) == 0,
             _T("Bug in wxWidgets: gripper should be at the bottom of Z-order") );
         ::DestroyWindow((HWND) m_hGripper);
         m_hGripper = 0;