On Thu, Mar 21, 2013 at 01:36:59PM +0100, Hans de Goede wrote: > With commit 81ed9d13 "virt_viewer_window_enter_fullscreen: Pass in monitor for > fullscreen window" we need a monitor number to determine where to move > the window when going fullscreen. > > Since the VirtViewerDisplay needs to know the fullscreen monitor number too, > to determine the fullscreen size it was being stored there. But we don't > always have a display, leading to errors like: > > (remote-viewer:7996): remote-viewer-CRITICAL **: > virt_viewer_display_get_monitor: assertion `VIRT_VIEWER_IS_DISPLAY(self)' > failed > > And to the monitor number not always being stored. This patchset fixes this > by storing the monitor number inside VirtViewerWindow, and passing it to > VirtViewerDisplay only when we've a display. Patch by itself seems ok, however this makes me wonder if all of this is not papering over that we are doing trying to do too much with VirtViewerWindow too early (ie before we have a display)? Christophe
Attachment:
pgp257xYIRoXc.pgp
Description: PGP signature
_______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
