Bug 20337

Summary: Implement CSS Animation and Transition Events
Product: WebKit Reporter: Dean Jackson <dino>
Component: DOMAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: dino, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
in progress patch
none
updated patch
none
working patch, with a test/example
none
now ready for review timothy: review+

Dean Jackson
Reported 2008-08-08 14:21:16 PDT
Need to implement the Event interfaces for CSS Animations and Transitions http://webkit.org/specs
Attachments
in progress patch (18.46 KB, patch)
2008-08-08 14:23 PDT, Dean Jackson
no flags
updated patch (37.87 KB, patch)
2008-08-08 14:47 PDT, Dean Jackson
no flags
working patch, with a test/example (43.23 KB, patch)
2008-08-08 15:17 PDT, Dean Jackson
no flags
now ready for review (47.35 KB, patch)
2008-08-08 15:26 PDT, Dean Jackson
timothy: review+
Dean Jackson
Comment 1 2008-08-08 14:23:11 PDT
Created attachment 22709 [details] in progress patch Still need to link up dom stuff. Won't build on non-mac platforms.
Dean Jackson
Comment 2 2008-08-08 14:47:23 PDT
Created attachment 22710 [details] updated patch still won't build on non-OSX
Dean Jackson
Comment 3 2008-08-08 15:17:33 PDT
Created attachment 22711 [details] working patch, with a test/example Just need to fix other platforms build.
Dean Jackson
Comment 4 2008-08-08 15:26:02 PDT
Created attachment 22712 [details] now ready for review Now should build on all platforms.
Timothy Hatcher
Comment 5 2008-08-08 20:48:32 PDT
Comment on attachment 22712 [details] now ready for review + if (eventType == EventNames::webkitTransitionEndEvent) { + element->dispatchWebKitTransitionEvent(eventType, name, elapsedTime); + } + else { + element->dispatchWebKitAnimationEvent(eventType, name, elapsedTime); + } The braces should be removed here. Otherwise looks good, and Hyatt looked over it earlier.
Dean Jackson
Comment 6 2008-08-11 10:25:12 PDT
Modified after review comments. Committed r35666 M WebCore/GNUmakefile.am M WebCore/bindings/js/JSEventCustom.cpp M WebCore/bindings/js/JSDOMWindowBase.h M WebCore/bindings/js/JSDOMWindowBase.cpp M WebCore/DerivedSources.make M WebCore/html/HTMLElement.cpp M WebCore/html/HTMLAttributeNames.in A WebCore/manual-tests/transition-events.html M WebCore/dom/Event.cpp A WebCore/dom/WebKitAnimationEvent.h M WebCore/dom/EventTargetNode.cpp A WebCore/dom/WebKitAnimationEvent.idl M WebCore/dom/Document.h A WebCore/dom/WebKitAnimationEvent.cpp M WebCore/dom/EventTargetNode.h M WebCore/dom/Event.h A WebCore/dom/WebKitTransitionEvent.idl A WebCore/dom/WebKitTransitionEvent.cpp M WebCore/dom/EventTarget.cpp A WebCore/dom/WebKitTransitionEvent.h M WebCore/WebCore.xcodeproj/project.pbxproj M WebCore/WebCore.pro M WebCore/ChangeLog M WebCore/WebCore.vcproj/WebCore.vcproj M WebCore/WebCoreSources.bkl M WebCore/page/AnimationController.cpp
Note You need to log in before you can comment on or make changes to this bug.