Fixing the WPT 'unset_context_name.html'
Categories
(Core :: DOM: Core & HTML, task, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox86 | --- | fixed |
People
(Reporter: timhuang, Assigned: timhuang)
References
Details
Attachments
(1 file)
The WPT unset_context_name.html is supposed to be the test for ensuring the browsing context's name will be reset when navigating across origins. However, the test was not correctly written and accidentally the test unset_context_name-1.html is running as the test. The unset_context_name-1.html is supposed to be the testing resources of the test 'unset_context_name.html'.
And the unset_context_name-1.html is running directly as the main testing page which would be given a UUID as its browsing context name in WPT. It means that this test is supposed to fail, but clearly, it's not right now. It passes for desktop because we enable 'privacy.window.name.update.enabled' and for android because of an issue that Bug 1687390 wants to resolve. We should fix this test.
Note that this test will start to fail if Bug 1687390 was resolved. It's because the browsing context name will be set properly and pass to the testing page. I've asked jgraham that if this is expected that the testing page would have the browsing context name set. And he said this is expected and it needs to use window.open() to open a new window if we want a clear browsing context.
|
Assignee | |
Comment 1•3 years ago
|
||
The WPT 'unset_context_name.html' doesn't do the test correctly. It was
using the wrong page 'unset_context_name-1.html' to do the test.
Althought, the test should be running by 'unset_context_name.html'
instead of 'unset_context_name-1.html'. This patch fixes this issue.
|
||
Updated•3 years ago
|
Pushed by tihuang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/54bdb171440d Fixing the WPT 'unset_context_name.html'. r=smaug
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/27263 for changes under testing/web-platform/tests
|
||
Comment 4•3 years ago
|
||
| bugherder | ||
Upstream PR merged by moz-wptsync-bot
Description
•