Bug 43792 – Accelerated transitions do not suspend/resume properly.

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Chris Marrin]

If you call AnimationController::suspendAnimations() while an accelerated transition is running, it does not suspend. When you call resumeAnimations() after the transition has finished, it will perform the transition again from the point at which suspend was called and then snap back to the start state briefly at the end.

This can be tested when https://bugs.webkit.org/show_bug.cgi?id=43733 is landed

rdar://problem/7616086

[Chris Marrin]

Created attachment 67108 [details]
Partial patch. Transitions pause and resume, but sometimes at end of transition, flashes back to start for an instant

This patch is just a checkpoint. The basic bug is fixed, but there is sometimes a flash back to the start frame at the end of the transition.

[Simon Fraser (smfr)]

Created attachment 68634 [details]
Main patch: unify transitions and animations in GraphicsLayer, and pause transitions.

[Simon Fraser (smfr)]

Comment on attachment 68634 [details]
Main patch: unify transitions and animations in GraphicsLayer, and pause transitions.

As Chris noted, with these changes, unpaused transitions can jump at the end. I'll fix that in a followup.

Comment on attachment 68634 [details]
Main patch: unify transitions and animations in GraphicsLayer, and pause transitions.

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

> WebCore/platform/graphics/GraphicsLayer.cpp:252
> +    return String::format("-|transition%d-", property);

Can you use %c instead?

> WebCore/platform/graphics/GraphicsLayer.h:295
> +    virtual void removeAnimation(const String& /*animationName*/)  { }

Extra space before the brace.

> WebCore/platform/graphics/mac/GraphicsLayerCA.h:275
> +    void setCAAnimationOnLayer(CAPropertyAnimation*, AnimatedPropertyID property, const String& animationName, int index, double timeOffset);
> +    bool removeCAAnimationFromLayer(AnimatedPropertyID property, const String& animationName, int index);
> +    void pauseCAAnimationOnLayer(AnimatedPropertyID property, const String& animationName, int index, double timeOffset);

I don’t know why you named the AnimatedPropertyID parameter in these declarations.

> WebCore/platform/graphics/mac/GraphicsLayerCA.h:340
> +        LayerPropertyAnimation(CAPropertyAnimation* caAnim, const String& animationName, AnimatedPropertyID property, int index, double timeOffset)

Could rename caAnim to caPropertyAnimation.

> WebCore/platform/graphics/mac/GraphicsLayerCA.mm:549
> +    NSString* animationID = animationIdentifier;

Star on the wrong side.

> WebCore/platform/graphics/mac/GraphicsLayerCA.mm:550
> +    CAAnimation* anim = [fromLayer animationForKey:animationID];

Ditto.

> WebCore/rendering/RenderLayerBacking.cpp:1238
> +// This is used for the 'freeze' api, for testing only.

API

[Simon Fraser (smfr)]

Comment on attachment 68634 [details]
Main patch: unify transitions and animations in GraphicsLayer, and pause transitions.

http://trac.webkit.org/changeset/68233

Keeping bug open to fix flash at end of resumed transition.

[Simon Fraser (smfr)]

Created attachment 68795 [details]
Patch

[Simon Fraser (smfr)]

http://trac.webkit.org/changeset/68323