Bug 136084

Summary: Add animationDidEnd callbacks on GraphicsLayer
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, esprehn+autocc, glenn, kondapallykalyan, sergio, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch thorton: review+

Description Simon Fraser (smfr) 2014-08-19 15:54:23 PDT
Add animationDidEnd callbacks on GraphicsLayer
Comment 1 Simon Fraser (smfr) 2014-08-19 15:56:50 PDT
Created attachment 236830 [details]
Patch
Comment 2 Tim Horton 2014-08-19 16:01:13 PDT
Comment on attachment 236830 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=236830&action=review

> Source/WebCore/platform/graphics/ca/mac/PlatformCALayerMac.mm:352
> +void PlatformCALayerMac::animationStarted(const String& animtionKey, CFTimeInterval beginTime)

anim*a*tion

> Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeContext.mm:137
> +    auto it = m_layersWithAnimations.find(layerID);

should we not remove the layer from m_layersWithAnimations if it doesn't have any more animations?
Comment 3 Simon Fraser (smfr) 2014-08-19 17:29:51 PDT
Created attachment 236837 [details]
Patch
Comment 4 Simon Fraser (smfr) 2014-08-21 15:44:08 PDT
https://trac.webkit.org/r172836