Bug in DOMMatrix in Chrome 61+

61 views
Skip to first unread message

Gerry Iles

unread,
Aug 21, 2017, 5:31:23 AM8/21/17
to Chromium-dev
I tracked down and reported a problem in Chrome's implementation of the DOMMatrix interface on Friday but I didn't set the "Component" sensibly and I think this may negatively affect it's visibility to the developers.  I wasn't really sure what to set it to and presumed I, or someone else, could change it later, but this doesn't seem to be the case (at least, I don't appear to be able to change it).  I don't really want to create a new bug that is basically a duplicate but with a sensible component so I'm posting here to get advice and/or bring the issue to someone's attention.


Basically, we are using a cross-browser wrapper class to set the transform used on an SVG use element and setting the m33 or m44 elements is being handled incorrectly.  If they are set to 0 or -0 then is2D is not set to false but if they are set to anything else then is2D is set to false.  These two elements should be testing against 1, not 0 or -0.  This incorrect value of is2D causes the toString function to generate a matrix3d() rather than a matrix() and this doesn't work for SVG use elements.  This is a pretty fundamental bug in the new implementation and I'm assuming the fix should be almost trivial so would really like to see it fixed before the release of 61.

So, can the component be changed to something sensible?  I'm still not really sure what it should be, there is a geometry related one but it doesn't mention the new DOMMatrix interface (or other, related, interfaces).  Is there any other way of bringing it to a relevant person's attention?

Thanks,
  Gerry

Fredrik Söderquist

unread,
Aug 21, 2017, 6:25:41 AM8/21/17
to gerry...@gmail.com, Chromium-dev
I set the Component to Blink>Geometry.


/fs
 

Thanks,
  Gerry

--
--
Chromium Developers mailing list: chromi...@chromium.org
View archives, change email options, or unsubscribe:
http://groups.google.com/a/chromium.org/group/chromium-dev
---
You received this message because you are subscribed to the Google Groups "Chromium-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to chromium-dev+unsubscribe@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/chromium-dev/43cf677c-c311-42a5-b8bc-5d49c9125016%40chromium.org.

Gerry Iles

unread,
Aug 21, 2017, 7:52:58 AM8/21/17
to Chromium-dev, gerry...@gmail.com
Thanks very much, (and for taking on the issue as well)...
Reply all
Reply to author
Forward
0 new messages