Tidy up XMLHttpRequest::endLoading().

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

Index: third_party/WebKit/Source/core/fetch/FetchUtils.h
diff --git a/third_party/WebKit/Source/core/fetch/FetchUtils.h b/third_party/WebKit/Source/core/fetch/FetchUtils.h
index 31657670ba0eb14af5982dcd8437dee9718669db..44e983bbb4e382da1c00e67233ee1f312dfe82fb 100644
--- a/third_party/WebKit/Source/core/fetch/FetchUtils.h
+++ b/third_party/WebKit/Source/core/fetch/FetchUtils.h
@@ -32,6 +32,10 @@ class CORE_EXPORT FetchUtils {
                                          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