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

disable browsing context name test #12929

Merged
merged 1 commit into from Aug 19, 2016
Merged
Changes from all commits
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

disable browsing context name test

  • Loading branch information
cbrewster committed Aug 18, 2016
commit 8db4c46c9a212eb283914a442769d6a07ec72b34
@@ -1,5 +1,3 @@
[browsing_context_name.html]
type: testharness
[Retaining window.name on history traversal]
expected: FAIL

disabled: see https://github.com/whatwg/html/issues/490

This comment has been minimized.

@KiChjang

KiChjang Aug 18, 2016

Member

I'm kinda curious, how is this related?

This comment has been minimized.

@metajack

metajack Aug 18, 2016

Contributor

The test looks for events for an iframe onload for each step. The initial frame has no src attribute, which means it will load about:blank. Based on whatwg/html#490, it looks like the behavior is unknown when the src changes before the onload event fires. That would make this test racy depending on the browser I'm guessing.

ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.