[virt-tools-list] [PATCH virt-viewer 03/12] kiosk: open a window on each client monitor

[Daniel P. Berrange]

On Mon, Jul 15, 2013 at 09:36:10PM +0200, Marc-André Lureau wrote:
> ---
>  src/virt-viewer-app.c | 22 +++++++++++++++++++++-
>  1 file changed, 21 insertions(+), 1 deletion(-)
> 
> diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c
> index 794dddd..5456b8d 100644
> --- a/src/virt-viewer-app.c
> +++ b/src/virt-viewer-app.c
> @@ -1235,6 +1235,26 @@ static void virt_viewer_app_usb_failed(VirtViewerSession *session G_GNUC_UNUSED,
>  }
>  
>  static void
> +virt_viewer_app_set_kiosk(VirtViewerApp *self, gboolean enabled)
> +{
> +    int i;
> +
> +    self->priv->kiosk = enabled;
> +    virt_viewer_app_set_fullscreen(self, enabled);
> +
> +    if (enabled) {
> +        for (i = 1; i < gdk_screen_get_n_monitors(gdk_screen_get_default()); i++) {
> +            if (virt_viewer_app_get_nth_window(self, i))
> +                continue;
> +
> +            VirtViewerWindow *win = virt_viewer_app_window_new(self, i);
> +            virt_viewer_window_show(win);
> +        }
> +    }
> +}

What's going to happen here with VNC which only has one monitor, or with
spice if they don't have the guest agent present to configure the guest
to use all monitors. Will we just get full screen, but empty windows ?

It would be desirable if the commit message was more verbose about the
behaviour of this change.

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|