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

Add wpt for soft navigation resource timing #40629

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

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jun 19, 2023

This CL adds a wpt that asserts a LCP entry and a resource timing entry
both of which correspond to an added image that caused a soft navigation
should have a navigation id that is the same as that of the
corresponding soft navigation entry.

Current this is true for LCP entry but not the resource timing entry.
Because the LCP entry is created after the soft navigation entry while
the resource timing entry is created before.

Change-Id: Ic1331a4502c332f09c413daf950504dc6d5bf01a
Reviewed-on: https://chromium-review.googlesource.com/4621233
WPT-Export-Revision: 6c3fb00fe981940419df42b9e17d278acc695122

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

This CL adds a wpt that asserts a LCP entry and a resource timing entry
both of which correspond to an added image that caused a soft navigation
should have a navigation id that is the same as that of the
corresponding soft navigation entry.

Current this is true for LCP entry but not the resource timing entry.
Because the LCP entry is created after the soft navigation entry while
the resource timing entry is created before.

Change-Id: Ic1331a4502c332f09c413daf950504dc6d5bf01a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants