[virt-tools-list] [PATCH virt-viewer] Remove gtk_window_present() call

Marc-André Lureau marcandre.lureau at gmail.com
Wed Mar 20 18:05:12 UTC 2013


gtk_window_present() may forcefully call gdk_window_show(), which will
call ShowWindow(). Although gdk call is not supposed to move the
window if it's already visible, it does restore the window position on
Vista+. For example, a snapped window will be moved back to its
previous position.

Gtk+ ShowWindow() is currently using SW_SHOWNOACTIVATE, it should
probably use SW_SHOWNA instead, but that didn't help anyway for a
snapped window.

Since virt_viewer_window_show() already ensure the window is visible,
I am not sure why gtk_window_present() is there in the first place, so
just remove it.

https://bugzilla.redhat.com/show_bug.cgi?id=912713
---
 src/virt-viewer-app.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c
index ff94c77..11d6985 100644
--- a/src/virt-viewer-app.c
+++ b/src/virt-viewer-app.c
@@ -674,7 +674,6 @@ display_show_hint(VirtViewerDisplay *display,
     } else if (hint & VIRT_VIEWER_DISPLAY_SHOW_HINT_READY) {
         virt_viewer_notebook_show_display(nb);
         virt_viewer_window_show(win);
-        gtk_window_present(virt_viewer_window_get_window(win));
     } else {
         if (win != self->priv->main_window &&
             g_getenv("VIRT_VIEWER_HIDE"))
-- 
1.8.1.1.439.g50a6b54




More information about the virt-tools-list mailing list