Last Comment Bug 848293 - Update AnimationEvent to be compatible with the spec
: Update AnimationEvent to be compatible with the spec
: dev-doc-complete
Product: Core
Classification: Components
Component: Layout (show other bugs)
: unspecified
: x86_64 Linux
: -- normal (vote)
: mozilla23
Assigned To: Olli Pettay [:smaug]
: Jet Villegas (:jet)
Depends on: 847589
Blocks: 868769
  Show dependency treegraph
Reported: 2013-03-06 03:57 PST by Olli Pettay [:smaug]
Modified: 2013-08-28 01:58 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (26.08 KB, patch)
2013-05-04 10:56 PDT, Olli Pettay [:smaug]
dbaron: review+
Details | Diff | Splinter Review
patch (26.15 KB, patch)
2013-05-05 06:22 PDT, Olli Pettay [:smaug]
no flags Details | Diff | Splinter Review

Description Olli Pettay [:smaug] 2013-03-06 03:57:31 PST
Better to do this after webidl-fication.
Comment 2 David Baron :dbaron: ⌚️UTC-8 2013-05-04 11:10:45 PDT
Comment on attachment 745567 [details] [diff] [review]

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.

Comment 3 Olli Pettay [:smaug] 2013-05-05 06:22:06 PDT
Created attachment 745639 [details] [diff] [review]
Comment 5 Phil Ringnalda (:philor) 2013-05-05 17:18:20 PDT

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