Closed
Bug 1514538
Opened 5 years ago
Closed 5 years ago
SVGMatrix and DOMMatrix are not interchangeable in Path2D.addPath
Categories
(Core :: SVG, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: vitalyx, Assigned: saschanaz)
Details
(Keywords: compat, parity-chrome, Whiteboard: [webcompat][wptsync upstream])
Attachments
(1 file)
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.
Updated•5 years ago
|
Component: Untriaged → SVG
Product: Firefox → Core
Updated•5 years ago
|
Updated•5 years ago
|
Whiteboard: [webcompat]
Comment 1•5 years ago
|
||
Migrating Webcompat whiteboard priorities to project flags. See bug 1547409.
Webcompat Priority: --- → ?
Comment 2•5 years ago
|
||
See bug 1547409. Migrating whiteboard priority tags to program flags.
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → saschanaz
Assignee | ||
Comment 3•5 years ago
|
||
Adds SVGMatrix toJSON to make it compatible with DOMMatrix2DInit.
Pushed by bzbarsky@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/efb3a42f50dc Support DOMMatrix2DInit for addPath r=bzbarsky
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/17857 for changes under testing/web-platform/tests
Whiteboard: [webcompat] → [webcompat][wptsync upstream]
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Comment 7•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox70:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
Upstream PR merged
You need to log in
before you can comment on or make changes to this bug.
Description
•