<div dir="ltr">Jonathon,<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Aug 12, 2014 at 8:31 PM, Jonathon Jongsma <span dir="ltr"><<a href="mailto:jjongsma@redhat.com" target="_blank">jjongsma@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">On Tue, 2014-08-12 at 18:03 +0200, Fabiano Fidêncio wrote:<br>

> We have to force displays to update geometry when the agent connects to<br>
> ensure the client will have the guest with the right resolution when the<br>
> guest has rebooted or the agent has crashed,<br>
<br>
</div>Does this fix a particular bug?  If so, it should be mentioned here.<br></blockquote><div><br></div><div>Thiis patch is not fixing any particular bug we have opened, but a "new" bug I found while testing stuff for <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1021841">https://bugzilla.redhat.com/show_bug.cgi?id=1021841</a></div>
<div>I don't think it's worthwhile mentioning it, but if you prefer, I can do :-)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div><div class="h5"><br>
> ---<br>
>  src/virt-viewer-session-spice.c | 4 ++++<br>
>  src/virt-viewer-session.c       | 4 ++++<br>
>  src/virt-viewer-session.h       | 1 +<br>
>  3 files changed, 9 insertions(+)<br>
><br>
> diff --git a/src/virt-viewer-session-spice.c b/src/virt-viewer-session-spice.c<br>
> index b6886be..a27d433 100644<br>
> --- a/src/virt-viewer-session-spice.c<br>
> +++ b/src/virt-viewer-session-spice.c<br>
> @@ -618,6 +618,10 @@ agent_connected_changed(SpiceChannel *cmain G_GNUC_UNUSED,<br>
>  {<br>
>      // this will force refresh of application menu<br>
>      g_signal_emit_by_name(self, "session-display-updated");<br>
> +<br>
> +    /* this will force update displays geometry when the agent has connected<br>
> +     * after the application (eg: rebooting the guest) */<br>
> +    virt_viewer_session_update_displays_geometry(VIRT_VIEWER_SESSION(self));<br>
>  }<br>
><br>
>  static void<br>
> diff --git a/src/virt-viewer-session.c b/src/virt-viewer-session.c<br>
> index c432719..d9c84a6 100644<br>
> --- a/src/virt-viewer-session.c<br>
> +++ b/src/virt-viewer-session.c<br>
> @@ -476,6 +476,10 @@ void virt_viewer_session_clear_displays(VirtViewerSession *session)<br>
>      session->priv->displays = NULL;<br>
>  }<br>
><br>
> +void virt_viewer_session_update_displays_geometry(VirtViewerSession *session)<br>
> +{<br>
> +    virt_viewer_session_on_monitor_geometry_changed(session, NULL);<br>
> +}<br>
><br>
><br>
>  void virt_viewer_session_close(VirtViewerSession *session)<br>
> diff --git a/src/virt-viewer-session.h b/src/virt-viewer-session.h<br>
> index 316fdf0..ba0f6b2 100644<br>
> --- a/src/virt-viewer-session.h<br>
> +++ b/src/virt-viewer-session.h<br>
> @@ -107,6 +107,7 @@ void virt_viewer_session_add_display(VirtViewerSession *session,<br>
>  void virt_viewer_session_remove_display(VirtViewerSession *session,<br>
>                                          VirtViewerDisplay *display);<br>
>  void virt_viewer_session_clear_displays(VirtViewerSession *session);<br>
> +void virt_viewer_session_update_displays_geometry(VirtViewerSession *session);<br>
><br>
>  void virt_viewer_session_close(VirtViewerSession* session);<br>
>  gboolean virt_viewer_session_open_fd(VirtViewerSession* session, int fd);<br>
<br>
<br>
</div></div>I don't love the name _update_displays_geometry(), but I can't really<br>
think of anything better at the moment.  I think the approach is<br>
basically correct (unsurprisingly, I suppose, since I suggested it), so<br>
ACK from me.<br>
<div class=""><div class="h5"><br>
_______________________________________________<br>
virt-tools-list mailing list<br>
<a href="mailto:virt-tools-list@redhat.com">virt-tools-list@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/virt-tools-list" target="_blank">https://www.redhat.com/mailman/listinfo/virt-tools-list</a></div></div></blockquote></div><br><br clear="all"><div>Best Regards,</div>-- <br>
<div>Fabiano Fidêncio</div>
</div></div>