Commit e3d6ee43 authored by Lucas Furukawa Gadani's avatar Lucas Furukawa Gadani Committed by Commit Bot

Only process frame-ancestors CSP for response checks.

Bug: 759184
Change-Id: I3cb3b30d7c52c6ce174833702057d9c323f688f1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1942870Reviewed-by: default avatarArthur Sonzogni <arthursonzogni@chromium.org>
Commit-Queue: Lucas Gadani <lfg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#720005}
parent db575e9c
...@@ -183,8 +183,8 @@ NavigationThrottle::ThrottleCheckResult AncestorThrottle::ProcessResponseImpl( ...@@ -183,8 +183,8 @@ NavigationThrottle::ThrottleCheckResult AncestorThrottle::ProcessResponseImpl(
// Evaluate whether the navigation should be allowed or blocked based on // Evaluate whether the navigation should be allowed or blocked based on
// existing content-security-policy on the response. // existing content-security-policy on the response.
if (base::FeatureList::IsEnabled( if (is_response_check && base::FeatureList::IsEnabled(
network::features::kOutOfBlinkFrameAncestors)) { network::features::kOutOfBlinkFrameAncestors)) {
if (network::mojom::ContentSecurityPolicyPtr policy = if (network::mojom::ContentSecurityPolicyPtr policy =
request->response()->head.content_security_policy) { request->response()->head.content_security_policy) {
// TODO(arthursonzogni): Remove content::ContentSecurityPolicy in favor of // TODO(arthursonzogni): Remove content::ContentSecurityPolicy in favor of
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment