Chromium Code Reviews| Index: Source/core/loader/DocumentThreadableLoader.cpp |
| diff --git a/Source/core/loader/DocumentThreadableLoader.cpp b/Source/core/loader/DocumentThreadableLoader.cpp |
| index 247f43c8bb743553d9e07826b945988685f3c213..42d67d7eca17cbc6e0847045e50490d669a0b9a6 100644 |
| --- a/Source/core/loader/DocumentThreadableLoader.cpp |
| +++ b/Source/core/loader/DocumentThreadableLoader.cpp |
| @@ -206,13 +206,18 @@ void DocumentThreadableLoader::redirectReceived(Resource* resource, ResourceRequ |
| // scheme and not contain the userinfo production. In addition, the redirect response must pass the access control check if the |
| // original request was not same-origin. |
| if (m_options.crossOriginRequestPolicy == UseAccessControl) { |
| + |
| + InspectorInstrumentation::didReceiveCORSRedirectResponse(m_document->frame(), resource->identifier(), m_document->frame()->loader()->documentLoader(), redirectResponse, 0); |
| + |
| bool allowRedirect = false; |
| + String accessControlErrorDescription; |
| if (m_simpleRequest) { |
| - String accessControlErrorDescription; |
| allowRedirect = SchemeRegistry::shouldTreatURLSchemeAsCORSEnabled(request.url().protocol()) |
| && request.url().user().isEmpty() |
| && request.url().pass().isEmpty() |
| && (m_sameOriginRequest || passesAccessControlCheck(redirectResponse, m_options.allowCredentials, securityOrigin(), accessControlErrorDescription)); |
| + } else { |
| + accessControlErrorDescription = "Cross Origin Requests with preflight cannot be redirected"; |
|
Mike West
2013/08/28 15:33:56
A bit more detail would be nice. How about somethi
ancilgeorge
2013/08/29 07:09:38
Done.
|
| } |
| if (allowRedirect) { |
| @@ -243,9 +248,12 @@ void DocumentThreadableLoader::redirectReceived(Resource* resource, ResourceRequ |
| makeCrossOriginAccessRequest(request); |
| return; |
| } |
| - } |
| - m_client->didFailRedirectCheck(); |
| + ResourceError error(errorDomainWebKitInternal, 0, redirectResponse.url().string(), accessControlErrorDescription); |
| + m_client->didFailAccessControlCheck(error); |
| + } else { |
| + m_client->didFailRedirectCheck(); |
| + } |
| request = ResourceRequest(); |
| } |