Prevent calling didReceiveData()/didFinishLoading() after didFailAccessControlCheck()

Source/core/loader/DocumentThreadableLoader.h

Index: Source/core/loader/DocumentThreadableLoader.h
diff --git a/Source/core/loader/DocumentThreadableLoader.h b/Source/core/loader/DocumentThreadableLoader.h
index 9f44f3a56334aa6878bbb54a37db8795acbc3278..a99a62b3db0f162a591b3a574d796b16250642cf 100644
--- a/Source/core/loader/DocumentThreadableLoader.h
+++ b/Source/core/loader/DocumentThreadableLoader.h
@@ -104,7 +104,10 @@ class DocumentThreadableLoader final : public ThreadableLoader, private Resource
         void loadActualRequest();
         // Clears m_actualRequest and reports access control check failure to
         // m_client.
-        void handlePreflightFailure(const String& url, const String& errorDescription);
+        // Call handleAccessControlCheckFailure() instead of
+        // m_client->didFailAccessControlCheck() from other places in

[tyoshino (SeeGerritForStatus), 2015/02/03 13:06:36]

what does "other" here mean?

[hiroshige, 2015/02/03 13:11:39]

I meant "anywhere outside handleAccessControlCheckFailure".
This is not so helpful so removed "from other places".

+        // DocumentThreadableLoader.
+        void handleAccessControlCheckFailure(const String& url, const String& errorDescription);
         // Investigates the response for the preflight request. If successful,
         // the actual request will be made later in handleSuccessfulFinish().
         void handlePreflightResponse(const ResourceResponse&);
@@ -154,6 +157,8 @@ class DocumentThreadableLoader final : public ThreadableLoader, private Resource
         // But this value is not the max number of total redirects allowed,
         // because same-origin redirects are not counted here.
         int m_corsRedirectLimit;
+
+        bool m_accessControlCheckFailed;
     };
 
 } // namespace blink