Closed
Bug 1436438
Opened 6 years ago
Closed 6 years ago
Remove the WebIDL methods for creating and mutating SVG path data
Categories
(Core :: SVG, enhancement)
Core
SVG
Tracking
()
RESOLVED
FIXED
mozilla60
People
(Reporter: jwatt, Assigned: jwatt)
References
Details
Attachments
(2 files)
30.10 KB,
patch
|
jwatt
:
review+
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
8.21 KB,
patch
|
jwatt
:
review+
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
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•6 years ago
|
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
Assignee | ||
Comment 2•6 years ago
|
||
Attachment #8949094 -
Flags: review+
Assignee | ||
Comment 3•6 years ago
|
||
Attachment #8949096 -
Flags: review+
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d2877c20431b https://hg.mozilla.org/mozilla-central/rev/3e3cfa85d219
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox60:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Comment 5•6 years ago
|
||
Adding Jason as this may be of interest to him.
Comment 6•6 years ago
|
||
Tracking since we're intending to uplift to beta.
status-firefox59:
--- → affected
tracking-firefox59:
--- → +
Comment 7•6 years ago
|
||
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+
Updated•6 years ago
|
Attachment #8949094 -
Flags: approval-mozilla-beta+
Comment hidden (obsolete) |
Comment 9•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/a9f01ec9c060 https://hg.mozilla.org/releases/mozilla-beta/rev/a9062e1bafb7
Comment 10•6 years 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
Comment 11•6 years ago
|
||
Sure but then you just need the same shim that you already need to support Chrome.
Comment 12•6 years 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.
Description
•