Closed Bug 847600 Opened 7 years ago Closed 7 years ago

Remove SVGEvent

Categories

(Core :: DOM: Events, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: smaug, Assigned: smaug)

References

Details

(Keywords: dev-doc-needed, site-compat)

Attachments

(2 files)

No description provided.
Summary: Paris bindings for SVGEvent and SVGZoomEvent → Paris binding for SVGEvent
It is not clear to me why we need SVGEvent interface at all.
Could we remove it?
Ping.

I'd like to avoid implementing useless events in Webidl.
The SVGEvent interface exists in SVG 1.0:

  http://www.w3.org/TR/SVG10/idl.html
  http://www.w3.org/TR/SVG10/script.html

but no longer exists in SVG 1.1:

  http://www.w3.org/TR/SVG11/idl.html
  http://www.w3.org/TR/SVG11/script.html

Seems like it can just go away. (Note the message types SVGLoad, SVGUnload, SVGAbort, SVGError, SVGResize and SVGScroll still exist, but we don't need the SVGEvent interface for them.)
Ok, thanks!
Attached patch patchSplinter Review
Assignee: nobody → bugs
Attachment #723111 - Flags: review?(jwatt)
Attachment #723111 - Flags: review?(jwatt) → review+
Summary: Paris binding for SVGEvent → Remove SVGEVent
Summary: Remove SVGEVent → Remove SVGEvent
https://hg.mozilla.org/mozilla-central/rev/d8648c78e3ad
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
I've added this bug to the compatibility doc. Please correct the info if wrong.
https://developer.mozilla.org/en-US/docs/Site_Compatibility_for_Firefox_22
You need to log in before you can comment on or make changes to this bug.