[Spice-devel] [spice-html5] display: Dump bitmap metadata in log_draw()
Christophe Fergeau
cfergeau at redhat.com
Tue Jul 17 08:26:07 UTC 2018
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 <cfergeau at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20180717/61eb400a/attachment.sig>
More information about the Spice-devel
mailing list