Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(550)

Unified Diff: third_party/WebKit/Source/core/loader/HttpEquiv.cpp

Issue 2415373002: Loading: bulk style errors fix in core/loader (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/loader/HttpEquiv.cpp
diff --git a/third_party/WebKit/Source/core/loader/HttpEquiv.cpp b/third_party/WebKit/Source/core/loader/HttpEquiv.cpp
index c5ff5547b4794a912f69a6ed6e23fbe0548ffaab..6a28964400bd7e05ee79bd3c2d9844590362ce9b 100644
--- a/third_party/WebKit/Source/core/loader/HttpEquiv.cpp
+++ b/third_party/WebKit/Source/core/loader/HttpEquiv.cpp
@@ -67,16 +67,17 @@ void HttpEquiv::processHttpEquivContentSecurityPolicy(
const AtomicString& content) {
if (document.importLoader())
return;
- if (equalIgnoringCase(equiv, "content-security-policy"))
+ if (equalIgnoringCase(equiv, "content-security-policy")) {
document.contentSecurityPolicy()->didReceiveHeader(
content, ContentSecurityPolicyHeaderTypeEnforce,
ContentSecurityPolicyHeaderSourceMeta);
- else if (equalIgnoringCase(equiv, "content-security-policy-report-only"))
+ } else if (equalIgnoringCase(equiv, "content-security-policy-report-only")) {
document.contentSecurityPolicy()->didReceiveHeader(
content, ContentSecurityPolicyHeaderTypeReport,
ContentSecurityPolicyHeaderSourceMeta);
- else
+ } else {
NOTREACHED();
+ }
}
void HttpEquiv::processHttpEquivAcceptCH(Document& document,
@@ -99,9 +100,10 @@ void HttpEquiv::processHttpEquivRefresh(Document& document,
UseCounter::count(document, UseCounter::MetaRefresh);
if (!document.contentSecurityPolicy()->allowInlineScript(
KURL(), "", ParserInserted, OrdinalNumber(), "",
- ContentSecurityPolicy::SuppressReport))
+ ContentSecurityPolicy::SuppressReport)) {
UseCounter::count(document,
UseCounter::MetaRefreshWhenCSPBlocksInlineScript);
+ }
document.maybeHandleHttpRefresh(content, Document::HttpRefreshFromMetaTag);
}
@@ -116,9 +118,10 @@ void HttpEquiv::processHttpEquivSetCookie(Document& document,
UseCounter::count(document, UseCounter::MetaSetCookie);
if (!document.contentSecurityPolicy()->allowInlineScript(
KURL(), "", ParserInserted, OrdinalNumber(), "",
- ContentSecurityPolicy::SuppressReport))
+ ContentSecurityPolicy::SuppressReport)) {
UseCounter::count(document,
UseCounter::MetaSetCookieWhenCSPBlocksInlineScript);
+ }
// Exception (for sandboxed documents) ignored.
document.setCookie(content, IGNORE_EXCEPTION);
« no previous file with comments | « third_party/WebKit/Source/core/loader/FrameLoader.cpp ('k') | third_party/WebKit/Source/core/loader/ImageLoader.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698