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
HTML: Expect salvegable to be set to false for pagehide test #11294
Conversation
Also rename the manual test. Part of #11269.
290722f
to
c5ec75c
Compare
@@ -4,9 +4,12 @@ | |||
onpagehide = function() { | |||
onpagehide = null; | |||
setTimeout(function() { | |||
parent.t.done() | |||
parent.t.step_func(function() { assert_unreached('setTimeout survived navigatoin'); }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
parent.t.unreached_func('setTimeout survived navigatoin')
would be more compact.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
@@ -2,4 +2,4 @@ | |||
<title>Document salvagable state after setting pagehide handler</title> | |||
<script>onpagehide = function() {setTimeout(function(){document.body.innerHTML = "PASS"}, 100)}</script> | |||
<p>Click the link below then navigate back to this page. Shortly after returning you should see the text "PASS"</p> | |||
<p><A href="manual-001-1.html">Click here</a> | |||
<p><a href="pagehide-manual-1.html">Click here</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clicking will now 404, is that part of the test, or accidental?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not 404, that file was renamed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, there's both pagehide-manual.html and pagehide-manual-1.html.
Also rename the manual test.
Part of #11269.
I will report an issue for whatwg/html about this test.