On Mon, Jul 16, 2018 at 10:03:23AM -0500, Jeremy White wrote: > Hey Christophe, > > On 07/12/2018 11:07 AM, Christophe Fergeau wrote: > > log_draw() is used to debug drawing commands by dumping drawing > > information to the JS console. This commit adds dumping of missing data > > for uncompressed bitmaps. The 'stride' parameter is important, as > > spice-html5 currently does not deal properly with such bitmaps. > > > > Signed-off-by: Christophe Fergeau <cferg...@redhat.com> > > --- > > display.js | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/display.js b/display.js > > index 7719b23..c55b686 100644 > > --- a/display.js > > +++ b/display.js > > @@ -769,6 +769,12 @@ SpiceDisplayConn.prototype.log_draw = function(prefix, > > draw) > > str += "; src_bitmap type " + > > draw.data.src_bitmap.descriptor.type + ", flags " + > > draw.data.src_bitmap.descriptor.flags; > > if (draw.data.src_bitmap.surface_id !== undefined) > > str += "; src_bitmap surface_id " + > > draw.data.src_bitmap.surface_id; > > + if (draw.data.src_bitmap.bitmap) > > + str += "; BITMAP format " + > > draw.data.src_bitmap.bitmap.format + > > + "; flags " + draw.data.src_bitmap.bitmap.format + > > That looks like a repeat of the format, rather than the flags.
Yup, this duplicates the width/heigh info, however this also has the stride which is not present in the other places. Showing this is the main goal of the patch, as I did not go as far as fixing the actual bug (too unfamiliar with js :-/) Christophe
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/spice-devel
