Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Record timing info for promise resolve/reject and callback #1400

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

noamr
Copy link

@noamr noamr commented Apr 18, 2024


Preview | Diff

@saschanaz
Copy link
Member

Maybe I lack the context, but having this in Web IDL covers a lot more than animation frames, right? Are the standards positions and implementation bugs actually cover the IDL layer change?

@noamr
Copy link
Author

noamr commented Apr 18, 2024

Maybe I lack the context, but having this in Web IDL covers a lot more than animation frames, right?

It does not, this is the shim between WebIDL and long animation frames. For long animation frames to report script timing it needs to know when particular scripts are enabled. This doesn't add anything web-observable to WebIDL directly.

Are the standards positions and implementation bugs actually cover the IDL layer change?

The explainers included the observable effect of this change as exposed by long animation frames from the start.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants