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

Clarify RT entry creation when iframe changes src #316

Closed
noamr opened this issue Jan 21, 2022 · 7 comments
Closed

Clarify RT entry creation when iframe changes src #316

noamr opened this issue Jan 21, 2022 · 7 comments

Comments

@noamr
Copy link
Contributor

noamr commented Jan 21, 2022

Take the following scenario:

  • Load an iframe with a src
  • When the iframe loads, the owner document changes the src to a different URL.

In WebKit/Chromium, there would be 1 resource entry in the performance timeline. In Gecko there would be 2.
The previous Resource Timing spec was a bit vague about it, where the scenario presented there was about internal iframe navigations.

I believe that WebKit/Chromium try to follow the spirit of the old spec, but that Gecko is doing the actual right thing (there's no point in hiding the resource entry for a resource loaded explicitly by the owner document).

This affects the iframe bit of whatwg/html#6542.

@noamr
Copy link
Contributor Author

noamr commented Jan 21, 2022

@yoavweiss
Copy link
Contributor

That makes sense. It's possible that Chromium's implementation of the restriction was over-zealous...

/cc @achristensen07

@achristensen07
Copy link
Contributor

Acknowledged. I don't have a strong preference either way

@yoavweiss
Copy link
Contributor

@noamr - can you file a crbug on this?

@noamr
Copy link
Contributor Author

noamr commented Jan 24, 2022

@noamr - can you file a crbug on this?

I was going to once I file the HTML PR, as right now this is unspecified. But I can start with a crbug.

@noamr
Copy link
Contributor Author

noamr commented Jan 25, 2022

@noamr
Copy link
Contributor Author

noamr commented Mar 30, 2022

whatwg/html#7531 and the new WPTS clear this up.

@noamr noamr closed this as completed Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants