Last Comment Bug 847600 - Remove SVGEvent
: Remove SVGEvent
Status: RESOLVED FIXED
: dev-doc-needed, site-compat
Product: Core
Classification: Components
Component: DOM: Events (show other bugs)
: unspecified
: x86_64 Linux
-- normal (vote)
: mozilla22
Assigned To: Olli Pettay [:smaug] (pto-ish for couple of days)
:
: Andrew Overholt [:overholt]
Mentors:
Depends on:
Blocks: 776864
  Show dependency treegraph
 
Reported: 2013-03-04 13:44 PST by Olli Pettay [:smaug] (pto-ish for couple of days)
Modified: 2013-05-16 11:53 PDT (History)
6 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (15.48 KB, patch)
2013-03-09 12:58 PST, Olli Pettay [:smaug] (pto-ish for couple of days)
jwatt: review+
Details | Diff | Splinter Review
patch (15.77 KB, patch)
2013-03-11 11:59 PDT, Olli Pettay [:smaug] (pto-ish for couple of days)
no flags Details | Diff | Splinter Review

Description User image Olli Pettay [:smaug] (pto-ish for couple of days) 2013-03-04 13:44:57 PST

    
Comment 1 User image Olli Pettay [:smaug] (pto-ish for couple of days) 2013-03-05 06:43:53 PST
It is not clear to me why we need SVGEvent interface at all.
Could we remove it?
Comment 2 User image Olli Pettay [:smaug] (pto-ish for couple of days) 2013-03-09 09:08:37 PST
Ping.

I'd like to avoid implementing useless events in Webidl.
Comment 3 User image Jonathan Watt [:jwatt] 2013-03-09 09:46:52 PST
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.)
Comment 4 User image Olli Pettay [:smaug] (pto-ish for couple of days) 2013-03-09 09:48:05 PST
Ok, thanks!
Comment 5 User image Olli Pettay [:smaug] (pto-ish for couple of days) 2013-03-09 12:58:26 PST
Created attachment 723111 [details] [diff] [review]
patch
Comment 6 User image Olli Pettay [:smaug] (pto-ish for couple of days) 2013-03-09 13:15:34 PST
https://tbpl.mozilla.org/?tree=Try&rev=84eab1736339
Comment 7 User image Olli Pettay [:smaug] (pto-ish for couple of days) 2013-03-11 11:59:00 PDT
Created attachment 723583 [details] [diff] [review]
patch

https://hg.mozilla.org/integration/mozilla-inbound/rev/d8648c78e3ad
Comment 8 User image Ryan VanderMeulen [:RyanVM] 2013-03-12 12:59:01 PDT
https://hg.mozilla.org/mozilla-central/rev/d8648c78e3ad
Comment 9 User image Kohei Yoshino [:kohei] 2013-03-29 10:17:36 PDT
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
Comment 10 User image :Ms2ger (⌚ UTC+1/+2) 2013-03-30 06:50:07 PDT
https://developer.mozilla.org/en-US/docs/DOM/Mozilla_event_reference too

Note You need to log in before you can comment on or make changes to this bug.