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

Consolidate paintTime and presentationTime in performance APIs #1110

Open
noamr opened this issue Nov 12, 2024 · 2 comments
Open

Consolidate paintTime and presentationTime in performance APIs #1110

noamr opened this issue Nov 12, 2024 · 2 comments

Comments

@noamr
Copy link

noamr commented Nov 12, 2024

Request for Mozilla Position on an Emerging Web Specification

Other information

This has been discussed at the WebPerfWG, see w3c/paint-timing#62

@sefeng211
Copy link
Member

Yeah, we talked this in WebPerf and we are positive about this.

Basically the changes on Firefox side will be

  1. We need to implement paintTime, and this is the timestamp after Update the rendering
  2. Need to change FCP's startTime to this paintTime, probably worth a developer notification because currently it's the timestamp before we do all the rAFs etc.
  3. We can decide how we want to expose presentationTime, it's optional though.
  4. I think LCP's renderTime will also be this paintTime?

@noamr
Copy link
Author

noamr commented Dec 3, 2024

Yeah, we talked this in WebPerf and we are positive about this.

Basically the changes on Firefox side will be

  1. We need to implement paintTime, and this is the timestamp after Update the rendering
  2. Need to change FCP's startTime to this paintTime, probably worth a developer notification because currently it's the timestamp before we do all the rAFs etc.
  3. We can decide how we want to expose presentationTime, it's optional though.
  4. I think LCP's renderTime will also be this paintTime?

LCP's renderTime would be the same as FCP startTime, so it would be presentationTime if it's implemented, and paintTime otherwise.

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

No branches or pull requests

2 participants