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

Spec-align resolving history handling during navigation #34747

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

Conversation

shanehandley
Copy link
Contributor

Makes use of the is_initial_about_blank property on Document in order to determine whether a navigation should create a history entry, or replace the current history entry.

This unfortunately introduces a new intermittent failure (/html/browsers/browsing-the-web/navigating-across-documents/initial-empty-document/iframe-nosrc.html, example). Initially I thought I should not push the change and open this PR, but it appears to belong in the family of intermittents that Gregory was looking into in #31973 and a little more spec alignment might be useful - but happy to close this if it's an issue.

Try


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • There are tests for these changes

Copy link
Member

@jdm jdm left a comment

Choose a reason for hiding this comment

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

Thanks for digging in here! Given my experience with #33087, I think incremental improvements like this are important even if they introduce new intermittent failures.

@jdm jdm added this pull request to the merge queue Dec 23, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 23, 2024
@shanehandley
Copy link
Contributor Author

Thank you! That's a lot of failures I wasn't expecting, I'm taking a look 👀

…tion

Makes use of the is_initial_about_blank property on Document in order to
determine whether the navigation should create a history entry, or replace
the current history entry.

Signed-off-by: Shane Handley <[email protected]>
@shanehandley shanehandley force-pushed the navigation-history-handling branch from 41d7a42 to 02b8e4e Compare December 24, 2024 02:14
@shanehandley
Copy link
Contributor Author

I've pushed a fix - I suspect I thought it was passing all tests becuase I confused it with another task 😬 sorry about that.

Here's an updated try: https://github.com/shanehandley/servo/actions/runs/12475391448

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

Successfully merging this pull request may close these issues.

2 participants