[Mono-bugs] [Bug 79326][Maj] Changed - X BadWindow errors in XplatUIX11.PerformNCCalc

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by toshok at ximian.com.

http://bugzilla.ximian.com/show_bug.cgi?id=79326

--- shadow/79326	2006-09-13 14:56:12.000000000 -0400
+++ shadow/79326.tmp.10752	2006-09-13 22:42:43.000000000 -0400
@@ -56,6 +56,21 @@
 seems to be the multiple RecreateHandle calls?
 
 ------- Additional Comments From toshok at ximian.com  2006-09-13 14:56 -------
 Created an attachment (id=17683)
 a tiny test which exhibits the problem
 
+
+------- Additional Comments From toshok at ximian.com  2006-09-13 22:42 -------
+so this is a combination of a couple of things.
+
+the control code was thinking it was done with recreation a little
+sooner than it was (since the recreation code was split into two
+parts, one in RecreateControl and another in WndProc (WM_DESTROY). 
+I'm going to commit a fix for this part.
+
+the XplatUIX11 code isn't disposing of Hwnd's like it needs to, which
+was causing events to come in (before we handled the DestroyNotify,
+but after the window had been destroyed on the server).  This is more
+difficult to fix, since we need to keep the window references around
+so that we can clear the focus/activated/caret windows when they go
+away.  this part needs more thought.