Lcp timing allow origin #37743

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.

Sign up for GitHub

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

Jump to bottom

Merged

sideshowbarker merged 3 commits into mdn:main from tunetheweb:lcp-timing-allow-origin

Jan 22, 2025

Contributor

tunetheweb commented Jan 21, 2025

Description

There have been some recent changes to renderTime for LCP and Element Timing and Chrome is changing this for Chrome 133.

Motivation

Additional details

See also:

Related issues and pull requests

mdn/browser-compat-data#25735

tunetheweb added 2 commits

January 21, 2025 18:28


          Update LCP render time docs

5f18e7a


          Update Element Timing too

8479c16

tunetheweb requested a review from a team as a code owner

January 21, 2025 18:40

tunetheweb requested review from sideshowbarker and removed request for a team

January 21, 2025 18:40

tunetheweb mentioned this pull request

Chrome 133 returns LCP and Element Timing renderTime for cross-origin media mdn/browser-compat-data#25735

Merged

github-actions bot added Content:WebAPI size/s labels

Contributor

github-actions bot commented Jan 21, 2025 •

edited

Loading

Preview URLs

External URLs (3)

URL: /en-US/docs/Web/API/PerformanceElementTiming/renderTime
Title: PerformanceElementTiming: renderTime property

Consider to expose decoding time for FCP and LCP? w3c/paint-timing#104 (1 time) (Note! This may be a new URL 👀)

URL: /en-US/docs/Web/API/LargestContentfulPaint
Title: LargestContentfulPaint

Consider to expose decoding time for FCP and LCP? w3c/paint-timing#104 (1 time) (Note! This may be a new URL 👀)

URL: /en-US/docs/Web/API/LargestContentfulPaint/renderTime
Title: LargestContentfulPaint: renderTime property

Consider to expose decoding time for FCP and LCP? w3c/paint-timing#104 (1 time) (Note! This may be a new URL 👀)

(comment last updated: 2025-01-22 01:42:24)

sideshowbarker reviewed

View reviewed changes

files/en-us/web/api/largestcontentfulpaint/index.md Outdated Show resolved Hide resolved

sideshowbarker reviewed

View reviewed changes

files/en-us/web/api/largestcontentfulpaint/index.md Outdated Show resolved Hide resolved

sideshowbarker reviewed

View reviewed changes

files/en-us/web/api/largestcontentfulpaint/index.md Outdated Show resolved Hide resolved

sideshowbarker reviewed

View reviewed changes

files/en-us/web/api/largestcontentfulpaint/rendertime/index.md Outdated Show resolved Hide resolved

sideshowbarker reviewed

View reviewed changes

files/en-us/web/api/largestcontentfulpaint/rendertime/index.md Outdated Show resolved Hide resolved

sideshowbarker reviewed

View reviewed changes

files/en-us/web/api/performanceelementtiming/rendertime/index.md Outdated Show resolved Hide resolved

sideshowbarker reviewed

View reviewed changes

files/en-us/web/api/performanceelementtiming/rendertime/index.md Outdated Show resolved Hide resolved

sideshowbarker reviewed

View reviewed changes

files/en-us/web/api/performanceelementtiming/rendertime/index.md Outdated Show resolved Hide resolved

sideshowbarker reviewed

View reviewed changes

files/en-us/web/api/performanceelementtiming/rendertime/index.md Outdated Show resolved Hide resolved


          Apply suggestions from code review

a28176d

sideshowbarker approved these changes

View reviewed changes

sideshowbarker merged commit 9070ad7 into mdn:main

9 checks passed

Contributor Author

tunetheweb commented Jan 22, 2025

Thanks for the edits and merging @sideshowbarker !

tunetheweb mentioned this pull request

Tweak LCP render time wording #41019

Merged

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

Labels

Content:WebAPI size/s