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

Unified Diff: trunk/Source/core/fetch/ResourceLoader.cpp

Issue 203593008: Revert 169496 "HTML Imports: Send credentials for same origin re..." (Closed) Base URL: svn://svn.chromium.org/blink/
Patch Set: Created 6 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
« no previous file with comments | « trunk/Source/core/fetch/ResourceLoader.h ('k') | trunk/Source/core/html/HTMLLinkElement.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/Source/core/fetch/ResourceLoader.cpp
===================================================================
--- trunk/Source/core/fetch/ResourceLoader.cpp (revision 169503)
+++ trunk/Source/core/fetch/ResourceLoader.cpp (working copy)
@@ -121,7 +121,7 @@
request.setReportLoadTiming(true);
ASSERT(m_state != Terminated);
ASSERT(!request.isNull());
- m_originalRequest = m_request = applyOptions(request);
+ m_originalRequest = m_request = request;
m_resource->updateRequest(request);
m_host->didInitializeResourceLoader(this);
}
@@ -153,6 +153,7 @@
m_loader = adoptPtr(blink::Platform::current()->createURLLoader());
ASSERT(m_loader);
blink::WrappedResourceRequest wrappedRequest(m_request);
+ wrappedRequest.setAllowStoredCredentials(m_options.allowCredentials == AllowStoredCredentials);
m_loader->loadAsynchronously(wrappedRequest, this);
}
@@ -173,7 +174,7 @@
if (m_loader)
m_loader->setDefersLoading(defers);
if (!defers && !m_deferredRequest.isNull()) {
- m_request = applyOptions(m_deferredRequest);
+ m_request = m_deferredRequest;
m_deferredRequest = ResourceRequest();
start();
}
@@ -262,7 +263,7 @@
{
RefPtr<ResourceLoader> protect(this);
- ResourceRequest& request(applyOptions(passedRequest.toMutableResourceRequest()));
+ ResourceRequest& request(passedRequest.toMutableResourceRequest());
ASSERT(!request.isNull());
const ResourceResponse& redirectResponse(passedRedirectResponse.toResourceResponse());
ASSERT(!redirectResponse.isNull());
@@ -271,7 +272,6 @@
return;
}
- applyOptions(request); // canAccessRedirect() can modify m_options so we should re-apply it.
m_host->redirectReceived(m_resource, redirectResponse);
m_resource->willSendRequest(request, redirectResponse);
if (request.isNull() || m_state == Terminated)
@@ -456,6 +456,7 @@
m_connectionState = ConnectionStateStarted;
blink::WrappedResourceRequest requestIn(m_request);
+ requestIn.setAllowStoredCredentials(m_options.allowCredentials == AllowStoredCredentials);
blink::WebURLResponse responseOut;
responseOut.initialize();
blink::WebURLError errorOut;
@@ -475,10 +476,4 @@
didFinishLoading(0, monotonicallyIncreasingTime(), encodedDataLength);
}
-ResourceRequest& ResourceLoader::applyOptions(ResourceRequest& request) const
-{
- request.setAllowStoredCredentials(m_options.allowCredentials == AllowStoredCredentials);
- return request;
}
-
-}
« no previous file with comments | « trunk/Source/core/fetch/ResourceLoader.h ('k') | trunk/Source/core/html/HTMLLinkElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698