[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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



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



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]