From 538ab4fca7822eadbe87987877a8f835964e10ae Mon Sep 17 00:00:00 2001 From: Alexander Larsson Date: Tue, 25 Oct 2011 15:36:13 +0200 Subject: [PATCH] win32: Remove most special casing of WINPOSCHANGED during modal ops There is no particular reason to special case this, we want to handle all sort of normal events. The only special thing we keep is that as an optimization we pump the message loop extra during a WINPOSCHANGED in a modal operation as that will cause us to repaint faster. Also, bump the arbitrary number of mainloop iterations for the timer. I don't see why we need it at all, but at least doing more than one iteration if needed should be nice. --- gdk/win32/gdkevents-win32.c | 233 ++++++++++++++++-------------------- 1 file changed, 101 insertions(+), 132 deletions(-) diff --git a/gdk/win32/gdkevents-win32.c b/gdk/win32/gdkevents-win32.c index 1ceeacc57b..bc2f8a5204 100644 --- a/gdk/win32/gdkevents-win32.c +++ b/gdk/win32/gdkevents-win32.c @@ -1654,7 +1654,7 @@ modal_timer_proc (HWND hwnd, UINT_PTR id, DWORD time) { - int arbitrary_limit = 1; + int arbitrary_limit = 10; while (_modal_operation_in_progress && g_main_context_pending (NULL) && @@ -2706,147 +2706,116 @@ gdk_event_translate (MSG *msg, buf))))), windowpos->cx, windowpos->cy, windowpos->x, windowpos->y)); - if (_modal_operation_in_progress) + /* Break grabs on unmap or minimize */ + if (windowpos->flags & SWP_HIDEWINDOW || + ((windowpos->flags & SWP_STATECHANGED) && IsIconic (msg->hwnd))) { - /* If position and size haven't changed, don't do anything */ - if ((windowpos->flags & SWP_NOMOVE) && - (windowpos->flags & SWP_NOSIZE)) - break; + if (pointer_grab != NULL) + { + if (pointer_grab->window == window) + gdk_pointer_ungrab (msg->time); + } - /* Once we've entered the moving or sizing modal loop, we won't - * return to the main loop until we're done sizing or moving. - */ + if (keyboard_grab->window == window) + gdk_keyboard_ungrab (msg->time); + } + + /* Send MAP events */ + if ((windowpos->flags & SWP_SHOWWINDOW) && + !GDK_WINDOW_DESTROYED (window)) + { + event = gdk_event_new (GDK_MAP); + event->any.window = window; + _gdk_win32_append_event (event); + } + + /* New size or position => configure event */ + if (!(windowpos->flags & SWP_NOCLIENTMOVE) || + !(windowpos->flags & SWP_NOCLIENTSIZE)) + { if (GDK_WINDOW_TYPE (window) != GDK_WINDOW_CHILD && + !IsIconic (msg->hwnd) && !GDK_WINDOW_DESTROYED (window)) - { - if (window->event_mask & GDK_STRUCTURE_MASK) - { - GDK_NOTE (EVENTS, g_print (" do magic")); - if (window->resize_count > 1) - window->resize_count -= 1; + handle_configure_event (msg, window); - handle_configure_event (msg, window); - g_main_context_iteration (NULL, FALSE); -#if 0 - /* Dispatch main loop - to realize resizes... */ - modal_timer_proc (msg->hwnd, msg->message, 0, msg->time); -#endif - /* Claim as handled, so that WM_SIZE and WM_MOVE are avoided */ - return_val = TRUE; - *ret_valp = 1; - } - } + if (window->extension_events != 0) + _gdk_device_wintab_update_window_coords (window); } - else /* !_modal_operation_in_progress */ + + if ((windowpos->flags & SWP_HIDEWINDOW) && + !GDK_WINDOW_DESTROYED (window)) { - /* Break grabs on unmap or minimize */ - if (windowpos->flags & SWP_HIDEWINDOW || - ((windowpos->flags & SWP_STATECHANGED) && IsIconic (msg->hwnd))) - { - if (pointer_grab != NULL) - { - if (pointer_grab->window == window) - gdk_pointer_ungrab (msg->time); - } + /* Send UNMAP events */ + event = gdk_event_new (GDK_UNMAP); + event->any.window = window; + _gdk_win32_append_event (event); - if (keyboard_grab->window == window) - gdk_keyboard_ungrab (msg->time); - } + /* Make transient parent the forground window when window unmaps */ + impl = GDK_WINDOW_IMPL_WIN32 (window->impl); - /* Send MAP events */ - if ((windowpos->flags & SWP_SHOWWINDOW) && - !GDK_WINDOW_DESTROYED (window)) - { - event = gdk_event_new (GDK_MAP); - event->any.window = window; - _gdk_win32_append_event (event); - } - - /* New size or position => configure event */ - if (!(windowpos->flags & SWP_NOCLIENTMOVE) || - !(windowpos->flags & SWP_NOCLIENTSIZE)) - { - if (GDK_WINDOW_TYPE (window) != GDK_WINDOW_CHILD && - !IsIconic (msg->hwnd) && - !GDK_WINDOW_DESTROYED (window)) - handle_configure_event (msg, window); - - if (window->extension_events != 0) - _gdk_device_wintab_update_window_coords (window); - } - - if ((windowpos->flags & SWP_HIDEWINDOW) && - !GDK_WINDOW_DESTROYED (window)) - { - /* Send UNMAP events */ - event = gdk_event_new (GDK_UNMAP); - event->any.window = window; - _gdk_win32_append_event (event); - - /* Make transient parent the forground window when window unmaps */ - impl = GDK_WINDOW_IMPL_WIN32 (window->impl); - - if (impl->transient_owner && - GetForegroundWindow () == GDK_WINDOW_HWND (window)) - SetForegroundWindow (GDK_WINDOW_HWND (impl->transient_owner)); - } - - /* Update window state */ - if (windowpos->flags & (SWP_STATECHANGED | SWP_SHOWWINDOW | SWP_HIDEWINDOW)) - { - GdkWindowState set_bits, unset_bits, old_state, new_state; - - old_state = window->state; - - set_bits = 0; - unset_bits = 0; - - if (IsWindowVisible (msg->hwnd)) - unset_bits |= GDK_WINDOW_STATE_WITHDRAWN; - else - set_bits |= GDK_WINDOW_STATE_WITHDRAWN; - - if (IsIconic (msg->hwnd)) - set_bits |= GDK_WINDOW_STATE_ICONIFIED; - else - unset_bits |= GDK_WINDOW_STATE_ICONIFIED; - - if (IsZoomed (msg->hwnd)) - set_bits |= GDK_WINDOW_STATE_MAXIMIZED; - else - unset_bits |= GDK_WINDOW_STATE_MAXIMIZED; - - - gdk_synthesize_window_state (window, unset_bits, set_bits); - - new_state = window->state; - - /* Whenever one window changes iconified state we need to also - * change the iconified state in all transient related windows, - * as windows doesn't give icons for transient childrens. - */ - if ((old_state & GDK_WINDOW_STATE_ICONIFIED) != - (new_state & GDK_WINDOW_STATE_ICONIFIED)) - do_show_window (window, (new_state & GDK_WINDOW_STATE_ICONIFIED)); - - - /* When un-minimizing, make sure we're stacked under any - transient-type windows. */ - if (!(old_state & GDK_WINDOW_STATE_ICONIFIED) && - (new_state & GDK_WINDOW_STATE_ICONIFIED)) - ensure_stacking_on_unminimize (msg); - } - - if (!(windowpos->flags & SWP_NOCLIENTSIZE)) - { - if (window->resize_count > 1) - window->resize_count -= 1; - } - - /* Claim as handled, so that WM_SIZE and WM_MOVE are avoided */ - return_val = TRUE; - *ret_valp = 0; + if (impl->transient_owner && + GetForegroundWindow () == GDK_WINDOW_HWND (window)) + SetForegroundWindow (GDK_WINDOW_HWND (impl->transient_owner)); } + + /* Update window state */ + if (windowpos->flags & (SWP_STATECHANGED | SWP_SHOWWINDOW | SWP_HIDEWINDOW)) + { + GdkWindowState set_bits, unset_bits, old_state, new_state; + + old_state = window->state; + + set_bits = 0; + unset_bits = 0; + + if (IsWindowVisible (msg->hwnd)) + unset_bits |= GDK_WINDOW_STATE_WITHDRAWN; + else + set_bits |= GDK_WINDOW_STATE_WITHDRAWN; + + if (IsIconic (msg->hwnd)) + set_bits |= GDK_WINDOW_STATE_ICONIFIED; + else + unset_bits |= GDK_WINDOW_STATE_ICONIFIED; + + if (IsZoomed (msg->hwnd)) + set_bits |= GDK_WINDOW_STATE_MAXIMIZED; + else + unset_bits |= GDK_WINDOW_STATE_MAXIMIZED; + + gdk_synthesize_window_state (window, unset_bits, set_bits); + + new_state = window->state; + + /* Whenever one window changes iconified state we need to also + * change the iconified state in all transient related windows, + * as windows doesn't give icons for transient childrens. + */ + if ((old_state & GDK_WINDOW_STATE_ICONIFIED) != + (new_state & GDK_WINDOW_STATE_ICONIFIED)) + do_show_window (window, (new_state & GDK_WINDOW_STATE_ICONIFIED)); + + + /* When un-minimizing, make sure we're stacked under any + transient-type windows. */ + if (!(old_state & GDK_WINDOW_STATE_ICONIFIED) && + (new_state & GDK_WINDOW_STATE_ICONIFIED)) + ensure_stacking_on_unminimize (msg); + } + + if (!(windowpos->flags & SWP_NOCLIENTSIZE)) + { + if (window->resize_count > 1) + window->resize_count -= 1; + } + + /* Call modal timer immediate so that we repaint faster after a resize. */ + if (_modal_operation_in_progress) + modal_timer_proc (0,0,0,0); + + /* Claim as handled, so that WM_SIZE and WM_MOVE are avoided */ + return_val = TRUE; + *ret_valp = 0; break; case WM_SIZING: