Bug 203075 – [Web Animations] Enable the Web Animations JavaScript API by default

• Status: RESOLVED
• Resolution: FIXED
• Component: Animations

[Antoine Quint]

[Web Animations] Enable the Web Animations JavaScript API by default

[Antoine Quint]

Created attachment 381175 [details]
Patch

[Antti Koivisto]

Comment on attachment 381175 [details]
Patch

Yay!

[WebKit Commit Bot]

Comment on attachment 381175 [details]
Patch

Clearing flags on attachment: 381175

Committed r251237: <https://trac.webkit.org/changeset/251237>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/56372801>

[Chris Dumez]

Comment on attachment 381175 [details]
Patch

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

> Source/WebCore/page/RuntimeEnabledFeatures.h:408
> +    bool m_areWebAnimationsEnabled { true };

WebAnimation is not well-behaved when it comes to the back/forward cache and I do not think we should enable this until it is. We have a feature to make the back/forward cache work all the time and enabling this is a step backward.

[Antoine Quint]

(In reply to Chris Dumez from comment #6)
> Comment on attachment 381175 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=381175&action=review
> 
> > Source/WebCore/page/RuntimeEnabledFeatures.h:408
> > +    bool m_areWebAnimationsEnabled { true };
> 
> WebAnimation is not well-behaved when it comes to the back/forward cache and
> I do not think we should enable this until it is. We have a feature to make
> the back/forward cache work all the time and enabling this is a step
> backward.

This is tracked by 203088.

[Geoffrey Garen]

Antoine, does that mean that you intend to fix bug 203088 soon? If so, how soon?

[Antoine Quint]

I do intend to fix it soon. I'm busy working on a performance regression with the "CSS Animations via Web Animations" flag (rdar://56017038), and as soon as I'm done with this, I will work on it.

[Dima Voytenko]

Could you please confirm the status of Web Animations? All the relevant issues seem to be RESOLVED FIXED, and I can even find some blog posts on the feature. But I'm still not seeing it enabled in the latest desktop Safari.

[Antoine Quint]

(In reply to Dima Voytenko from comment #10)
> Could you please confirm the status of Web Animations? All the relevant
> issues seem to be RESOLVED FIXED, and I can even find some blog posts on the
> feature. But I'm still not seeing it enabled in the latest desktop Safari.

Web Animations has been enabled for the first time in a shipping version of Safari this week with Safari 13.1 for macOS Catalina 10.15.4. There is also a newer version of Safari 12 for macOS High Sierra and macOS Mojave, but I'm not sure what the version number is.

If you have the latest and greatest shipping Safari but somehow Web Animations is still disabled, you most likely disabled it manually yourself under Develop > Experimental Features. You can reset all of those flags using the "Reset All to Defaults" item at the bottom of the experimental features menu.