Closed Bug 1133173 Opened 9 years ago Closed 8 years ago

remove SVGSVGElement.getElementById

Categories

(Core :: SVG, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: heycam, Unassigned)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
The SVG WG resolved to remove SVGSVGElement.getElementById for being silly and likely unused.

r?longsonr for the removal, r?bz for the IDL change DOM peer review.
Attachment #8564477 - Flags: review?(longsonr)
Attachment #8564477 - Flags: review?(bzbarsky)
Comment on attachment 8564477 [details] [diff] [review]
patch

r=me on the IDL change, but are we _very_ sure it's unused?  I bet it's used; we should gather some telemetry.
Attachment #8564477 - Flags: review?(bzbarsky) → review+
Perhaps you should create a pref (or prefs) and make all the changes conditional on it/them.
Comment on attachment 8564477 [details] [diff] [review]
patch

Announce on the newsgroup and then I think we should put this under a pref and disable on nightly only as a first step.
Attachment #8564477 - Flags: review?(longsonr) → review-
For this one, I really think a use counter of some sort would be helpful...
https://lists.w3.org/Archives/Public/www-svg/2016Jul/0005.html
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Updated the site compat doc accordingly.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: