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

CSP: Use policy container for policies forced by CSPEE #27800

Merged
merged 1 commit into from Mar 18, 2021

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Feb 26, 2021

Now that we can deliver Content Security Policies from the browser to
the renderer in the policy container, we don't need anymore the
navigation parameter forced_content_security_policies for CSP forced
by CSPEE.

Bug: 1149272
Change-Id: I348e8a99609fad5a9cd4f273c385c828e278b629
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2704862
Reviewed-by: Mike West <mkwst@chromium.org>
Reviewed-by: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Titouan Rigoudy <titouan@chromium.org>
Commit-Queue: Antonio Sartori <antoniosartori@chromium.org>
Cr-Commit-Position: refs/heads/master@{#864178}

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.

@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-2704862 branch 7 times, most recently from beb6395 to 8c9206f Compare March 8, 2021 07:23
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-2704862 branch 8 times, most recently from f125dc1 to 6098991 Compare March 17, 2021 13:28
Now that we can deliver Content Security Policies from the browser to
the renderer in the policy container, we don't need anymore the
navigation parameter forced_content_security_policies for CSP forced
by CSPEE.

Bug: 1149272
Change-Id: I348e8a99609fad5a9cd4f273c385c828e278b629
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2704862
Reviewed-by: Mike West <mkwst@chromium.org>
Reviewed-by: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Titouan Rigoudy <titouan@chromium.org>
Commit-Queue: Antonio Sartori <antoniosartori@chromium.org>
Cr-Commit-Position: refs/heads/master@{#864178}
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