Bug 183370 – [Web Animations] Add a new runtime flag to control whether CSS Animations and CSS Transitions should run using the Web Animations timeline

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

[Antoine Quint]

We need a new runtime flag to control whether CSS Animations and CSS Transitions should run using the Web Animations timeline or using CSSAnimationController.

[Antoine Quint]

<rdar://problem/38180729>

[Antoine Quint]

Created attachment 335100 [details]
Patch

[Dean Jackson]

Comment on attachment 335100 [details]
Patch

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

You should check with Weinig here.

> Source/WebKit/UIProcess/API/C/WKPreferences.cpp:527
> +void WKPreferencesSetCSSAnimationsAndCSSTransitionsBackedByWebAnimationsEnabled(WKPreferencesRef preferencesRef, bool flag)
> +{
> +    toImpl(preferencesRef)->setCSSAnimationsAndCSSTransitionsBackedByWebAnimationsEnabled(flag);
> +}
> +
> +bool WKPreferencesGetCSSAnimationsAndCSSTransitionsBackedByWebAnimationsEnabled(WKPreferencesRef preferencesRef)
> +{
> +    return toImpl(preferencesRef)->cssAnimationsAndCSSTransitionsBackedByWebAnimationsEnabled();
> +}
> +

Don't need this. Remove it.

> Source/WebKit/UIProcess/API/C/WKPreferencesRefPrivate.h:125
> +// Defaults to false
> +WK_EXPORT void WKPreferencesSetCSSAnimationsAndCSSTransitionsBackedByWebAnimationsEnabled(WKPreferencesRef, bool flag);
> +WK_EXPORT bool WKPreferencesGetCSSAnimationsAndCSSTransitionsBackedByWebAnimationsEnabled(WKPreferencesRef);
> +

Ditto.

> Source/WebKit/UIProcess/API/Cocoa/WKPreferences.mm:979
> +- (void)_setCSSAnimationsAndCSSTransitionsBackedByWebAnimationsEnabled:(BOOL)enabled
> +{
> +    _preferences->setCSSAnimationsAndCSSTransitionsBackedByWebAnimationsEnabled(enabled);
> +}
> +

Actually I'm not sure if you need any of this other than the .yaml change.

> Source/WebKit/WebProcess/InjectedBundle/InjectedBundle.cpp:196
> +    if (preference == "WebKitCSSAnimationsAndCSSTransitionsBackedByWebAnimationsEnabled")
> +        RuntimeEnabledFeatures::sharedFeatures().setCSSAnimationsAndCSSTransitionsBackedByWebAnimationsEnabled(enabled);

Maybe I misunderstand the code, but I thought you didn't need to do this explicitly because the .yaml is generating code for it.

[Tim Horton]

Comment on attachment 335100 [details]
Patch

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

>> Source/WebKit/UIProcess/API/C/WKPreferences.cpp:527
>> +
> 
> Don't need this. Remove it.

Some clients still use this API

>> Source/WebKit/WebProcess/InjectedBundle/InjectedBundle.cpp:196
>> +        RuntimeEnabledFeatures::sharedFeatures().setCSSAnimationsAndCSSTransitionsBackedByWebAnimationsEnabled(enabled);
> 
> Maybe I misunderstand the code, but I thought you didn't need to do this explicitly because the .yaml is generating code for it.

Maybe plumbing to overrideBoolPreferenceForTestRunner isn't generated yet?

[WebKit Commit Bot]

Comment on attachment 335100 [details]
Patch

Clearing flags on attachment: 335100

Committed r229334: <https://trac.webkit.org/changeset/229334>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.