Changeset 9101 for trunk/src/user32/win32wbase.cpp

Timestamp:

Aug 23, 2002, 5:06:01 PM (23 years ago)

Author:

sandervl

Message:

Ugly hack added to work around crash in PM when child window calls DestroyWindow for parent or owner in WM_DESTROY handler (solution: postpone DestroyWindow for parent/owner)

File:

• trunk/src/user32/win32wbase.cpp (modified) (5 diffs)

trunk/src/user32/win32wbase.cpp

@@ -1 @@
-/* $Id: win32wbase.cpp,v 1.335 2002-08-14 10:37:44 sandervl Exp $ */
+/* $Id: win32wbase.cpp,v 1.336 2002-08-23 15:06:01 sandervl Exp $ */
 /*
  * Win32 Window Base Class for OS/2
@@ -149 +149 @@
   fParentChange    = FALSE;
   fDestroyWindowCalled = FALSE;
+  fChildDestructionInProgress = FALSE;
   fTaskList        = FALSE;
   fParentDC        = FALSE;
@@ -924 +925 @@
 
     if(fDestroyWindowCalled == FALSE)
-    {//this window was destroyed because DestroyWindow was called for it's parent
+    {//this window was destroyed because DestroyWindow was called for its parent
      //so: send a WM_PARENTNOTIFY now as that hasn't happened yet
         if((getStyle() & WS_CHILD) && !(getExStyle() & WS_EX_NOPARENTNOTIFY))
@@ -2683 +2684 @@
 //******************************************************************************
 //Also destroys all the child windows (destroy children first, parent last)
+//TODO: Don't rely on PM to do the right thing. Send WM_(NC)DESTROY &
+//      destroy children ourselves (see Wine)
 //******************************************************************************
 BOOL Win32BaseWindow::DestroyWindow()
@@ -2760 +2763 @@
   
     fDestroyWindowCalled = TRUE;
-    return OSLibWinDestroyWindow(OS2HwndFrame);
+
+//hack alert; PM crashes if child calls DestroyWindow for parent/owner in WM_DESTROY
+//            handler; must postpone it
+    if(fChildDestructionInProgress) {
+        dprintf(("Postponing parent destruction because of dying child"));
+        OSLibPostMessageDirect(OS2HwndFrame, WIN32APP_POSTPONEDESTROY, 0, 0);
+        return TRUE;
+    }
+
+    BOOL fOldChildDestructionInProgress = FALSE;
+    if(GetParent()) {
+        Win32BaseWindow *window = Win32BaseWindow::GetWindowFromHandle(GetParent());
+        if(window) {
+            fOldChildDestructionInProgress = window->IsChildDestructionInProgress();
+            window->SetChildDestructionInProgress(TRUE);
+            RELEASE_WNDOBJ(window);
+        }
+    }
+//hack end
+
+    BOOL ret = OSLibWinDestroyWindow(OS2HwndFrame);
+
+//hack alert; PM crashes if child calls DestroyWindow for parent/owner in WM_DESTROY
+//            handler; must postpone it
+    if(GetParent()) {
+        Win32BaseWindow *window = Win32BaseWindow::GetWindowFromHandle(GetParent());
+        if(window) {
+            window->SetChildDestructionInProgress(fOldChildDestructionInProgress);
+            RELEASE_WNDOBJ(window);
+        }
+    }
+//hack end
+    return ret;
 }
 //******************************************************************************