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

[Gecko Bug 1574155] [wdspec] Use different top-level domains for cross-site iframe tests. #18495

Merged
merged 1 commit into from Aug 23, 2019

Conversation

moz-wptsync-bot
Copy link
Collaborator

Using a different subdomain as host for an iframe source doesn't actually
qualify for a cross-site origin. Instead the TLD has to be different.

Differential Revision: https://phabricator.services.mozilla.com/D42213

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1574155
gecko-commit: a61a256c7eafbbbc11b57ed0d8f0e1a1e0144cae
gecko-integration-branch: autoland
gecko-reviewers: webdriver-reviewers, ato

Using a different subdomain as host for an iframe source doesn't actually
qualify for a cross-site origin. Instead the TLD has to be different.

Differential Revision: https://phabricator.services.mozilla.com/D42213

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1574155
gecko-commit: a61a256c7eafbbbc11b57ed0d8f0e1a1e0144cae
gecko-integration-branch: autoland
gecko-reviewers: webdriver-reviewers, ato
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

Already reviewed downstream.

@jgraham
Copy link
Contributor

jgraham commented Aug 23, 2019

Just modifies too many tests for the CI to cope.

@jgraham jgraham merged commit 84b8df0 into master Aug 23, 2019
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1574155 branch August 23, 2019 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants