1514538 - SVGMatrix and DOMMatrix are not interchangeable in Path2D.addPath

Status: RESOLVED FIXED

(Core :: SVG, defect, P3)

Description

[Vitaly Kravchenko]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/71.0.3578.98 Safari/537.36

Steps to reproduce:

const p1 = new Path2D();
const p2 = new Path2D();
const m = new DOMMatrix();
p1.addPath(p2, m);


Actual results:

TypeError: Argument 2 of Path2D.addPath does not implement interface SVGMatrix.


Expected results:

Should have worked with no error.
Both SVGMatrix and DOMMatrix have the same constructor and are interchangeable here in Chrome and Safari, but not Firefox.

Comment 1

[Firefox Bug Husbandry Bot]

Migrating Webcompat whiteboard priorities to project flags. See bug 1547409.

Comment 2

[Firefox Bug Husbandry Bot]

See bug 1547409. Migrating whiteboard priority tags to program flags.

Comment 3

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1514538 - Support DOMMatrix2DInit for addPath

Adds SVGMatrix toJSON to make it compatible with DOMMatrix2DInit.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/efb3a42f50dc
Support DOMMatrix2DInit for addPath r=bzbarsky

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/17857 for changes under testing/web-platform/tests

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 7

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/efb3a42f50dc

Comment 8

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged