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

Do not use plugin-types for trusted-types WPTs #27305

Merged
merged 1 commit into from Jan 26, 2021

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jan 25, 2021

Some web platform tests for trusted types trigger a violation of the
CSP directive plugin-types to verify that a CSP violation trigger. The
plugin-types directive is specified with an invalid (hence in practice
empty) value. With the removal of the 'plugin-types' directive
implementations, these tests will break.

This CL converts those tests to trigger violations of the 'object-src'
directive instead.

Bug: 1168001
Change-Id: I042a7c600bc5c7caeb6dd87ff94b181b5fb66b6b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2646825
Commit-Queue: Antonio Sartori <antoniosartori@chromium.org>
Reviewed-by: Daniel Vogelheim <vogelheim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#847151}

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 Chromium project.

Some web platform tests for trusted types trigger a violation of the
CSP directive plugin-types to verify that a CSP violation trigger. The
plugin-types directive is specified with an invalid (hence in practice
empty) value. With the removal of the 'plugin-types' directive
implementations, these tests will break.

This CL converts those tests to trigger violations of the 'object-src'
directive instead.

Bug: 1168001
Change-Id: I042a7c600bc5c7caeb6dd87ff94b181b5fb66b6b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2646825
Commit-Queue: Antonio Sartori <antoniosartori@chromium.org>
Reviewed-by: Daniel Vogelheim <vogelheim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#847151}
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

3 participants