Skip to content

[Gecko Bug 1973182] trusted-types-navigation.html: Add tests for the case when the default policy throws or makes the URL invalid. #53284

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

moz-wptsync-bot
Copy link
Collaborator

Note that these tests are failing in Chromium and WebKit too, whereas
browsers should really report a CSP violation per the spec:
https://w3c.github.io/trusted-types/dist/spec/#require-trusted-types-for-pre-navigation-check

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1973182
gecko-commit: 5fe53e282b683922b1ba520700e9edf04f9d9e8b
gecko-reviewers: smaug

…t policy throws or makes the URL invalid.

Note that these tests are failing in Chromium and WebKit too, whereas
browsers should really report a CSP violation per the spec:
https://w3c.github.io/trusted-types/dist/spec/#require-trusted-types-for-pre-navigation-check

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1973182
gecko-commit: 5fe53e282b683922b1ba520700e9edf04f9d9e8b
gecko-reviewers: smaug
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.

The review process for this patch is being conducted in the Firefox project.

…e case when the default policy throws or makes the URL invalid. " for causing lint failures @ navigation-support.js

This reverts commit 4b9953661993be40e49abfa054ad99fab67e16fb.

gecko-commit: e362324cadfd67032f8c5f6c3f7f6bcd3f8fd49b
gecko-reviewers: smaug
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.

3 participants