stop listening to events on clients that are being unmanaged before generating new...

diff --git a/openbox/client.c b/openbox/client.c
index a65e7400313367609d03a8f92eab07494350423c..76db97bc82b0da54721fe384e795f9123796b6d2 100644 (file)
--- a/openbox/client.c
+++ b/openbox/client.c
@@ -506,6 +506,10 @@ void client_unmanage(ObClient *self)
 
     g_assert(self != NULL);
 
+    /* we dont want events no more. do this before hiding the frame so we
+       don't generate more events */
+    XSelectInput(ob_display, self->window, NoEventMask);
+
     frame_hide(self->frame);
     /* sync to send the hide to the server quickly, and to get back the enter
        events */
@@ -524,9 +528,6 @@ void client_unmanage(ObClient *self)
     /* remove the window from our save set */
     XChangeSaveSet(ob_display, self->window, SetModeDelete);
 
-    /* we dont want events no more */
-    XSelectInput(ob_display, self->window, NoEventMask);
-
     /* update the focus lists */
     focus_order_remove(self);