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

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

Issue 2645043004: Migrate expensive referrer CHECK to DCHECK in FrameFetchContext (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/loader/FrameFetchContext.cpp
diff --git a/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp b/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp
index 9cfa91a3c0ad2382dc369a6fe010894e4352f515..1974f3c60da7c784f159546aa0d78f6b3f7ad546 100644
--- a/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp
+++ b/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp
@@ -241,11 +241,11 @@ void FrameFetchContext::addAdditionalRequestHeaders(ResourceRequest& request,
m_document->outgoingReferrer()));
request.addHTTPOriginIfNeeded(m_document->getSecurityOrigin());
} else {
- CHECK_EQ(SecurityPolicy::generateReferrer(request.getReferrerPolicy(),
- request.url(),
- request.httpReferrer())
- .referrer,
- request.httpReferrer());
+ DCHECK_EQ(SecurityPolicy::generateReferrer(request.getReferrerPolicy(),
+ request.url(),
+ request.httpReferrer())
+ .referrer,
+ request.httpReferrer());
request.addHTTPOriginIfNeeded(request.httpReferrer());
}
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698