Tidy up XMLHttpRequest::endLoading().

third_party/WebKit/Source/core/fetch/FetchUtils.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef FetchUtils_h
 #define FetchUtils_h
 
 #include "core/CoreExport.h"
 #include "wtf/Allocator.h"
 #include "wtf/Forward.h"
@@ skipping 14 matching lines @@
   static bool isForbiddenMethod(const String& method);
   static bool isUsefulMethod(const String& method) {
     return !isForbiddenMethod(method);
   }
   static bool isForbiddenHeaderName(const String& name);
   static bool isForbiddenResponseHeaderName(const String& name);
   static bool isSimpleOrForbiddenRequest(const String& method,
                                          const HTTPHeaderMap&);
   static AtomicString normalizeMethod(const AtomicString& method);
   static String normalizeHeaderValue(const String& value);
+
+  static bool isSuccessfulStatusCode(int status) {

[tyoshino (SeeGerritForStatus), 2017/01/05 05:54:24]

how about using the term defined in the Fetch Standard?

[200, 299] is named "ok status".

[sof, 2017/01/05 08:11:03]

I consider "successful" the better term, but switched regardless.

+    return status >= 200 && status < 300;
+  }
 };
 
 }  // namespace blink
 
 #endif