[virt-tools-list] [PATCH virt-viewer 1/2] virt-viewer-display-spice: Skip monitor info in fullscreen-auto-conf mode

Hans de Goede hdegoede at redhat.com
Fri Mar 22 16:52:32 UTC 2013


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 at redhat.com>
>
> 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 at redhat.com>
> Acked-by: Marc-André Lureau <marcandre.lureau at redhat.com>
> ---
>   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 *
>




More information about the virt-tools-list mailing list