Bug 110057

Summary: [GTK][AC] Implement keyframe animations with clutter ac backend
Product: WebKit Reporter: ChangSeok Oh <kevin.cs.oh>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: gustavo, mrobinson, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Bug Depends on:    
Bug Blocks: 105699    
Attachments:
Description Flags
Patch
none
Patch none

ChangSeok Oh
Reported 2013-02-17 10:58:30 PST
This bug is for bringing accelearted animation with clutter to webkitgtk port.
Attachments
Patch (21.80 KB, patch)
2013-02-17 11:19 PST, ChangSeok Oh
no flags
Patch (21.98 KB, patch)
2013-02-19 07:42 PST, ChangSeok Oh
no flags
ChangSeok Oh
Comment 1 2013-02-17 11:19:08 PST
Gustavo Noronha (kov)
Comment 2 2013-02-19 06:37:18 PST
Comment on attachment 188770 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=188770&action=review > Source/WebCore/platform/graphics/clutter/GraphicsLayerClutter.cpp:775 > + // If any matrix is singular, CA won't animate it correctly. So fall back to software animation CA reference.
Gustavo Noronha (kov)
Comment 3 2013-02-19 06:37:37 PST
Comment on attachment 188770 [details] Patch Err, actually, there was a nit, so cq-
ChangSeok Oh
Comment 4 2013-02-19 07:42:52 PST
ChangSeok Oh
Comment 5 2013-02-19 07:45:33 PST
Comment on attachment 188770 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=188770&action=review Thank you for the review :) >> Source/WebCore/platform/graphics/clutter/GraphicsLayerClutter.cpp:775 >> + // If any matrix is singular, CA won't animate it correctly. So fall back to software animation > > CA reference. Updated this comment.
WebKit Review Bot
Comment 6 2013-02-19 09:03:14 PST
Comment on attachment 189089 [details] Patch Clearing flags on attachment: 189089 Committed r143343: <http://trac.webkit.org/changeset/143343>
WebKit Review Bot
Comment 7 2013-02-19 09:03:18 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.