Bug 257862 – accelerated animations not running at 120fps on pro motion devices

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: Animations

[Liam DeBeasi]

Created attachment 466640 [details]
Code reproduction

Accelerated animations are not running at 120fps on Pro Motion iOS devices.

Steps to reproduce:

1. Open code reproduction on a Pro Motion iOS device.
2. Tap "Open Overlay". Observe that the action sheet opens.
3. Tap outside of the action sheet. Observe that the action sheet closes.
4. Compare the smoothness of the animation with an action sheet found on iOS by opening the "Calendar" app.
5. Tap the "+" button to open the "New Event" modal.
6. Type something into the "Title" field.
7. Tap "Cancel". Observe that an action sheet opens, and the animation appears to be much smoother than the web animation.

Expected behavior:

I expect accelerated animations to run at 120fps on devices that support it.

Actual beahvior:

The web animation appears to only run at 60fps.

Other information:

- I tested this on an iPhone 13 Pro running iOS 16.5

[Dean Jackson]

We would animate purely accelerated animations at 120Hz. But I think the problem is we don't animate custom properties at that rate.

[Dean Jackson]

rdar://110670324

[Liam DeBeasi]

Created attachment 466680 [details]
Code reproduction with explicit values

Thanks for taking a look. I'm still able to reproduce the reported behavior even when using explicit values for each keyframe. I attached a new reproduction.

[Liam DeBeasi]

Created attachment 466871 [details]
Code reproduction with CSS Animations

I created another reproduction using CSS Animations, so this behavior does not appear to be limited to the Web Animations API. This reproduction also avoids custom properties.

Is there anything else I can do to help isolate this?

[Ahmad Saleem]

@Antoine - does below commit address this?

https://github.com/WebKit/WebKit/commit/69f1d447193caa27a7ce017a41f19d724520282e

[Antoine Quint]

I believe the fix for bug 267541 will address the latest variation (dated 2023-06029) of the reproduction posted by Liam where custom properties are not the values being interpolated. I'll try to check within the next couple of days, I have a few things preventing me from checking just now.

[Antoine Quint]

I can confirm that the translation animations run at 120Hz in all three demos.

*** This bug has been marked as a duplicate of bug 267541 ***