Hi, Both patches look good, ACK series. For those reading along, and also wondering why a patch of me is in Marc-André's patch-set, this set was first discussed here: https://bugzilla.redhat.com/show_bug.cgi?id=918997 And I attached my related patch there. Regards, Hans On 03/22/2013 02:14 PM, Marc-André Lureau wrote:
From: Hans de Goede <hdegoede@xxxxxxxxxx> When we are in fullscreen-auto-conf virt-viewer-session-spice sends a monitor-info message to the agent with the exact client monitor info, and virt-viewer-display-spice should not override that. Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> Acked-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx> --- src/virt-viewer-display-spice.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/virt-viewer-display-spice.c b/src/virt-viewer-display-spice.c index fb1f789..4ce8641 100644 --- a/src/virt-viewer-display-spice.c +++ b/src/virt-viewer-display-spice.c @@ -262,8 +262,15 @@ fullscreen_changed(VirtViewerApp *app, GParamSpec *pspec G_GNUC_UNUSED, VirtViewerDisplaySpice *self) { - self->priv->auto_resize = virt_viewer_app_get_fullscreen(app) ? - AUTO_RESIZE_FULLSCREEN : AUTO_RESIZE_ALWAYS; + if (virt_viewer_app_get_fullscreen(app)) { + gboolean auto_conf; + g_object_get(app, "fullscreen-auto-conf", &auto_conf, NULL); + if (auto_conf) + self->priv->auto_resize = AUTO_RESIZE_NEVER; + else + self->priv->auto_resize = AUTO_RESIZE_FULLSCREEN; + } else + self->priv->auto_resize = AUTO_RESIZE_ALWAYS; } GtkWidget *
_______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list