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

history.go(-1) in an iframe doesn't appear to be firing onunload in WPT html/browsers/browsing-the-web/navigating-across-documents/003.html and /004.html #25067

Open
pshaughn opened this issue Dec 3, 2019 · 1 comment

Comments

@pshaughn
Copy link
Member

@pshaughn pshaughn commented Dec 3, 2019

This test is broken up into different iframe pages so they can interact with history. 003-2.html posts a message that the parent document sees, then tries to go back in history. Its unload handler sets location = "003-3.html", but the parent never gets a message from 003-3.html. The obvious explanation is that the unload isn't firing, but of course the test has enough moving parts that there could be another explanation.

@pshaughn pshaughn changed the title history..go(-1) in an iframe doesn't appear to be firing onunload in WPT html/browsers/browsing-the-web/navigating-across-documents-003.html history.go(-1) in an iframe doesn't appear to be firing onunload in WPT html/browsers/browsing-the-web/navigating-across-documents-003.html Dec 3, 2019
@pshaughn pshaughn changed the title history.go(-1) in an iframe doesn't appear to be firing onunload in WPT html/browsers/browsing-the-web/navigating-across-documents-003.html history.go(-1) in an iframe doesn't appear to be firing onunload in WPT html/browsers/browsing-the-web/navigating-across-documents/003.html and /004.html Dec 3, 2019
@pshaughn
Copy link
Member Author

@pshaughn pshaughn commented Dec 3, 2019

004.html differs from 003.html slightly but the same net result happens.

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

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.