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

Remove new browsing context/Document referrer assignment #10883

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

Conversation

domfarolino
Copy link
Member

@domfarolino domfarolino commented Dec 23, 2024

This implements part of the proposed plan in #10099, which notes that only Chrome implements step 16.1 of "create a new browsing context and document", which assigns the brand new about:blank Document's referrer to its creator's origin, if it exists. This PR updates the spec to match Safari and Firefox, and Chrome's behavior will change accordingly.

  • At least two implementers are interested (and none opposed):
    • Chrome
    • Safari (already implements)
    • Gecko (already implements)
  • Tests are written and can be reviewed and commented upon at:
  • Implementation bugs are filed:
    • Chromium:
  • The top of this comment includes a clear commit message to use.

(See WHATWG Working Mode: Changes for more details.)


/document-sequences.html ( diff )

@domfarolino domfarolino added topic: frames/navigables/browsing contexts do not merge yet Pull request must not be merged per rationale in comment labels Dec 23, 2024
@domfarolino domfarolino marked this pull request as ready for review December 23, 2024 16:21
@domfarolino domfarolino requested a review from domenic December 23, 2024 16:22
@domfarolino
Copy link
Member Author

PTAL, but I've added the DNM label on this until:

  • I figure out which tests to already exists to link here, or which ones I need to implement
  • File a Chromium bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge yet Pull request must not be merged per rationale in comment topic: frames/navigables/browsing contexts
Development

Successfully merging this pull request may close these issues.

1 participant