On 02/03/2016 06:11 AM, Pavel Grunt wrote:
> Avoid calling gtk_widget_queue_resize() and emiting
> the "display-desktop-resize" signal.
> The only user of the properties is virt_viewer_display_spice_set_desktop()
> which will call the function and emit the signal after setting both
> "desktop-width" and "desktop-height" properties.
It just occurred to me if this will have any impact with VNC display.
Have you tested this path?
> ---
> src/virt-viewer-display.c | 8 ++------
> 1 file changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/src/virt-viewer-display.c b/src/virt-viewer-display.c
> index 4bfa29b..654cada 100644
> --- a/src/virt-viewer-display.c
> +++ b/src/virt-viewer-display.c
> @@ -301,14 +301,10 @@ virt_viewer_display_set_property(GObject *object,
>
> switch (prop_id) {
> case PROP_DESKTOP_WIDTH:
> - virt_viewer_display_set_desktop_size(display,
> - g_value_get_int(value),
> - priv->desktopHeight);
> + priv->desktopWidth = g_value_get_int(value);
> break;
> case PROP_DESKTOP_HEIGHT:
> - virt_viewer_display_set_desktop_size(display,
> - priv->desktopWidth,
> - g_value_get_int(value));
> + priv->desktopHeight = g_value_get_int(value);
> break;
> case PROP_NTH_DISPLAY:
> priv->nth_display = g_value_get_int(value);
>
Acked-by: Eduardo Lima (Etrunko) <[email protected]>
--
Eduardo de Barros Lima (Etrunko)
Software Engineer - RedHat
[email protected]
_______________________________________________
virt-tools-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-tools-list
