RESOLVED FIXED 138931
Support unprefixed animation event types 
https://bugs.webkit.org/show_bug.cgi?id=138931
Summary Support unprefixed animation event types
Dean Jackson
Reported 2014-11-20 12:45:06 PST
Support unprefixed animation event types
Attachments
Patch (61.77 KB, patch)
2014-11-20 13:11 PST, Dean Jackson 		no flags
DetailsFormatted DiffDiff
Patch (62.41 KB, patch)
2014-11-20 14:50 PST, Dean Jackson 		no flags
DetailsFormatted DiffDiff
Patch (62.87 KB, patch)
2014-11-20 15:21 PST, Dean Jackson 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Dean Jackson
Comment 1 2014-11-20 12:45:29 PST
<rdar://problem/18943167>
Dean Jackson
Comment 2 2014-11-20 12:47:19 PST
The approach here is to follow what transitions does: listener only for old event name -> fire old event name listener for both old and new event name -> fire new event name listener only for new event name -> fire new event name That way pages that were trying to anticipate the new event name won't get duplicate events.
Dean Jackson
Comment 3 2014-11-20 13:11:38 PST
Created attachment 241973 [details] Patch
Simon Fraser (smfr)
Comment 4 2014-11-20 13:47:21 PST
Comment on attachment 241973 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=241973&action=review > Source/WebCore/dom/AnimationEvent.cpp:35 > + , elapsedTime(0.0) Just 0. > Source/WebCore/dom/AnimationEvent.cpp:40 > + : m_elapsedTime(0.0) Just 0.
Dean Jackson
Comment 5 2014-11-20 14:50:26 PST
Created attachment 241986 [details] Patch
Dean Jackson
Comment 6 2014-11-20 15:21:36 PST
Created attachment 241991 [details] Patch
Dean Jackson
Comment 7 2014-11-20 15:27:25 PST
Committed r176423: <http://trac.webkit.org/changeset/176423>
Zoltan Horvath
Comment 8 2014-12-08 09:54:23 PST
Comment on attachment 241991 [details] Patch Clearing flags - pulling it out from the review queue. It was landed earlier in r176423.
Note You need to log in before you can comment on or make changes to this bug.