Tidy up XMLHttpRequest::endLoading().

third_party/WebKit/Source/core/fetch/CrossOriginAccessControl.cpp

 /*
  * Copyright (C) 2008 Apple Inc. All Rights Reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 263 matching lines @@
 
   return true;
 }
 
 bool passesPreflightStatusCheck(const ResourceResponse& response,
                                 String& errorDescription) {
   // CORS preflight with 3XX is considered network error in
   // Fetch API Spec: https://fetch.spec.whatwg.org/#cors-preflight-fetch
   // CORS Spec: http://www.w3.org/TR/cors/#cross-origin-request-with-preflight-0
   // https://crbug.com/452394
-  if (response.httpStatusCode() < 200 || response.httpStatusCode() >= 300) {
+  int statusCode = response.httpStatusCode();
+  if (!FetchUtils::isOkStatus(statusCode)) {
     errorDescription = "Response for preflight has invalid HTTP status code " +
-                       String::number(response.httpStatusCode());
+                       String::number(statusCode);
     return false;
   }
 
   return true;
 }
 
 bool passesExternalPreflightCheck(const ResourceResponse& response,
                                   String& errorDescription) {
   AtomicString result =
       response.httpHeaderField(HTTPNames::Access_Control_Allow_External);
@@ skipping 130 matching lines @@
     //
     // This is equivalent to the step 2 in
     // https://fetch.spec.whatwg.org/#http-network-or-cache-fetch
     if (options.credentialsRequested == ClientDidNotRequestCredentials)
       options.allowCredentials = DoNotAllowStoredCredentials;
   }
   return true;
 }
 
 }  // namespace blink