Closed Bug 1436438 Opened 3 years ago Closed 3 years ago

Remove the WebIDL methods for creating and mutating SVG path data

Categories

(Core :: SVG, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla60
Tracking Status
firefox59 + fixed
firefox60 --- fixed

People

(Reporter: jwatt, Assigned: jwatt)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

Spinning this off from bug 1388931 comment 35 to land a minimal subset of the r+'ed patches there in order to remove the WebIDL methods for creating and mutating SVG path data. We can finish off the rest of that bug later.
Component: Layout → SVG
Pushed by jwatt@jwatt.org:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d2877c20431b
part 1 - Remove the tests that test SVG path data DOM interfaces. r=longsonr
https://hg.mozilla.org/integration/mozilla-inbound/rev/3e3cfa85d219
part 2 - Remove the WebIDL methods for creating and mutating SVG path data. r=longsonr,baku
https://hg.mozilla.org/mozilla-central/rev/d2877c20431b
https://hg.mozilla.org/mozilla-central/rev/3e3cfa85d219
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Adding Jason as this may be of interest to him.
Tracking since we're intending to uplift to beta.
Comment on attachment 8949096 [details] [diff] [review]
part 2 - Remove the WebIDL methods for creating and mutating SVG path data. r=longsonr,baku

OK to uplift. This should land for 59 beta 11.
Attachment #8949096 - Flags: approval-mozilla-beta+
Shouldn't the FF59 developer release notes mention this change (it broke my web apps) ?
https://developer.mozilla.org/en-US/Firefox/Releases/59
Sure but then you just need the same shim that you already need to support Chrome.
Not the same shim, an updated one because parts of the old API remain, unlike in Chrome.
https://github.com/progers/pathseg

This is why breaking changes would be appreciated in the release notes and more than three weeks before release.
You need to log in before you can comment on or make changes to this bug.