Closed Bug 848293 Opened 7 years ago Closed 7 years ago
Event to be compatible with the spec
Better to do this after webidl-fication.
Attachment #745567 - Flags: review?(dbaron)
Comment on attachment 745567 [details] [diff] [review] patch Could you add an assertion at the start of ElementAnimations::GetPositionInIteration that !aAnimation == !aEa && !aAnimation == !aEventsToDispatch. >+is(e.elapsedTime, "0.5"); I'm surprised the 0.5 is in quotes. Does that work? And again, it would be good to remove initAnimationEvent in a followup patch. r=dbaron
Attachment #745567 - Flags: review?(dbaron) → review+
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
Doc updated/created: https://developer.mozilla.org/en-US/docs/Web/API/AnimationEvent https://developer.mozilla.org/en-US/docs/Web/API/AnimationEvent.AnimationEvent https://developer.mozilla.org/en-US/docs/Web/API/AnimationEvent.pseudoElement https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/23 I didn't changed: https://developer.mozilla.org/en-US/docs/Web/API/AnimationEvent.initAnimationEvent because the new fourth optional arguments never went into a release as the method has been removed by bug 868751 (that is still in progress to be documented).
You need to log in before you can comment on or make changes to this bug.