Closed Bug 868751 Opened 11 years ago Closed 11 years ago

Remove support for document.createEvent("AnimationEvent"), document.createEvent("TransitionEvent"), AnimationEvent.initAnimationEvent, and TransitionEvent.initTransitionEvent

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: smaug, Assigned: dbaron)

References

Details

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

Attachments

(1 file)

Need to check what Webkit, Trident and Blink have and also do some
search whether the feature is used in the web pages or script libraries.
No longer depends on: 848291
Depends on: 848291
At the very least, the event type argument to document.createEvent() in WebKit / blink isn't compatible, or wasn't until very recently.  I think we should just remove and see if we get feedback; I have a patch to do so.
Assignee: nobody → dbaron
Comment on attachment 746671 [details] [diff] [review]
Remove support for obsolete document.createEvent("AnimationEvent"), document.createEvent("TransitionEvent"), AnimationEvent.initAnimationEvent, and TransitionEvent.initTransitionEvent.

Note that this comment:
>+  // NEW EVENT TYPES SHOULD NOT BE ADDED HERE; THEY SHOULD USE ONLY EVENT
>+  // CONSTRUCTORS
was added based on an in-person discussion I had with annevk.
Comment on attachment 746671 [details] [diff] [review]
Remove support for obsolete document.createEvent("AnimationEvent"), document.createEvent("TransitionEvent"), AnimationEvent.initAnimationEvent, and TransitionEvent.initTransitionEvent.

I guess we can do this this way, although I'd probably leave init*Event method to
.idl files and make them [noscript].
Attachment #746671 - Flags: review?(bugs) → review+
(In reply to Olli Pettay [:smaug] from comment #6)
> I guess we can do this this way, although I'd probably leave init*Event
> method to
> .idl files and make them [noscript].

Why would you want to do that?
oh, right, creating these events in C++ outside libxul becomes hard anyway. So, removing
init* from .idl makes sense.
Keywords: dev-doc-needed
Summary: Investigate if initTransitionEvent method could be removed → Remove support for document.createEvent("AnimationEvent"), document.createEvent("TransitionEvent"), AnimationEvent.initAnimationEvent, and TransitionEvent.initTransitionEvent
https://hg.mozilla.org/mozilla-central/rev/a20ea0d494a0
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
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_23
Keywords: site-compat
OS: Linux → All
Hardware: x86_64 → All
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.