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

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

Issue 1801513003: Cross origin requests to same origin should match Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed failing CORS tests Created 4 years, 9 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/LinkLoader.cpp
diff --git a/third_party/WebKit/Source/core/loader/LinkLoader.cpp b/third_party/WebKit/Source/core/loader/LinkLoader.cpp
index 4f4e00e3e00c8190c3dfaa0acdede072e495ed21..b6fc15740cc9735f9e41655c0d8beb2f38f1f483 100644
--- a/third_party/WebKit/Source/core/loader/LinkLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/LinkLoader.cpp
@@ -277,8 +277,7 @@ static Resource* preloadIfNeeded(const LinkRelAttribute& relAttribute, const KUR
FetchRequest linkRequest(resourceRequest, FetchInitiatorTypeNames::link);
linkRequest.setPriority(document.fetcher()->loadPriority(resourceType, linkRequest));
- if (crossOrigin != CrossOriginAttributeNotSet)
- linkRequest.setCrossOriginAccessControl(document.getSecurityOrigin(), crossOrigin);
+ linkRequest.setCrossOriginAccessControl(document.getSecurityOrigin(), crossOrigin);
Settings* settings = document.settings();
if (settings && settings->logPreload())
document.addConsoleMessage(ConsoleMessage::create(OtherMessageSource, DebugMessageLevel, String("Preload triggered for " + href.host() + href.path())));
@@ -341,8 +340,7 @@ bool LinkLoader::loadLink(const LinkRelAttribute& relAttribute, CrossOriginAttri
UseCounter::count(document, UseCounter::LinkRelPrefetch);
FetchRequest linkRequest(ResourceRequest(document.completeURL(href)), FetchInitiatorTypeNames::link);
- if (crossOrigin != CrossOriginAttributeNotSet)
- linkRequest.setCrossOriginAccessControl(document.getSecurityOrigin(), crossOrigin);
+ linkRequest.setCrossOriginAccessControl(document.getSecurityOrigin(), crossOrigin);
setResource(LinkFetchResource::fetch(Resource::LinkPrefetch, linkRequest, document.fetcher()));
}

Powered by Google App Engine
This is Rietveld 408576698