Fail preflight request when redirect is received

Source/core/loader/DocumentThreadableLoader.cpp

Index: Source/core/loader/DocumentThreadableLoader.cpp
diff --git a/Source/core/loader/DocumentThreadableLoader.cpp b/Source/core/loader/DocumentThreadableLoader.cpp
index c497a11efbbd8a6f6c5da97955cb00b4ac7fe57c..4ff4893a80151d7be4717ce44f1cdafbb36f2a2c 100644
--- a/Source/core/loader/DocumentThreadableLoader.cpp
+++ b/Source/core/loader/DocumentThreadableLoader.cpp
@@ -314,6 +314,19 @@ void DocumentThreadableLoader::redirectReceived(Resource* resource, ResourceRequ
 
     RefPtr<DocumentThreadableLoader> protect(this);
 
+    if (m_actualRequest) {
+        reportResponseReceived(resource->identifier(), redirectResponse);
+
+        clearResource();
+        request = ResourceRequest();

[Nate Chapin, 2015/08/31 20:05:26]

Is this line still necessary? I thought we had gotten rid of all logic that
required nulling a ResourceRequest to cancel it.

[tyoshino (SeeGerritForStatus), 2015/09/01 08:05:46]

Ah, yeah. You suggested this before in
https://codereview.chromium.org/643733003#msg4.

Seems it's time to try removing this. I'll do it in a separate CL.

+
+        m_requestStartedSeconds = 0.0;
+
+        handlePreflightFailure(redirectResponse.url().string(), "Response for preflight is invalid (redirect)");
+
+        return;
+    }
+
     if (m_redirectMode == WebURLRequest::FetchRedirectModeManual) {
         // We use |m_redirectMode| to check the original redirect mode.
         // |request| is a new request for redirect. So we don't set the redirect