Remove the WebIDL methods for creating and mutating SVG path data

RESOLVED FIXED in Firefox 59

Status

()

RESOLVED FIXED
a year ago
11 months ago

People

(Reporter: jwatt, Assigned: jwatt)

Tracking

(Blocks: 1 bug)

unspecified
mozilla60
Points:
---

Firefox Tracking Flags

(firefox59+ fixed, firefox60 fixed)

Details

Attachments

(2 attachments)

(Assignee)

Description

a year ago
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.
(Assignee)

Updated

a year ago
Component: Layout → SVG

Comment 1

a year ago
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
(Assignee)

Comment 2

a year ago
Created attachment 8949094 [details] [diff] [review]
part 1 - Remove the tests that test SVG path data DOM interfaces. r=longsonr
Attachment #8949094 - Flags: review+
(Assignee)

Comment 3

a year ago
Created attachment 8949096 [details] [diff] [review]
part 2 - Remove the WebIDL methods for creating and mutating SVG path data. r=longsonr,baku
Attachment #8949096 - Flags: review+

Comment 4

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d2877c20431b
https://hg.mozilla.org/mozilla-central/rev/3e3cfa85d219
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox60: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Adding Jason as this may be of interest to him.
Tracking since we're intending to uplift to beta.
status-firefox59: --- → affected
tracking-firefox59: --- → +
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+
Attachment #8949094 - Flags: approval-mozilla-beta+
Comment hidden (obsolete)

Comment 10

11 months ago
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.

Comment 12

11 months ago
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.