[WIP] WebFonts cache-aware timeout adaption

third_party/WebKit/Source/platform/exported/WebURLError.cpp

Index: third_party/WebKit/Source/platform/exported/WebURLError.cpp
diff --git a/third_party/WebKit/Source/platform/exported/WebURLError.cpp b/third_party/WebKit/Source/platform/exported/WebURLError.cpp
index 3654cae0c736c5442eb870a4fffa03303eb9d270..942bb680ddb6ed3740a1b83a76e53ab49523a042 100644
--- a/third_party/WebKit/Source/platform/exported/WebURLError.cpp
+++ b/third_party/WebKit/Source/platform/exported/WebURLError.cpp
@@ -30,6 +30,7 @@
 
 #include "public/platform/WebURLError.h"
 
+#include "platform/network/NetworkUtils.h"
 #include "platform/network/ResourceError.h"
 #include "platform/weborigin/KURL.h"
 
@@ -65,4 +66,14 @@ WebURLError::operator ResourceError() const {
   return resourceError;
 }
 
+WebURLError WebURLError::cacheMissError() {
+  WebURLError error;
+  error.reason = NetworkUtils::cacheMissErrorCode();

[kinuko, 2016/10/21 10:35:13]

This rather looks a little awkward layering workaround to me, I think we can try
adding deps to net/base/ (or just net/base/net_errors.h to be more specific)
from public/platform or Source/platform and ask Elliott about it.  We wouldn't
want to have uncontrolled dependency but net/base dependency from platform feels
reasonable to me.

[kinuko, 2016/10/22 04:43:01]

Hmm... looking into this a little further, actually I feel adding one more
boolean here was better aligned at least in the current code structure. As you
mentioned somewhere WebURLError's reason code is not meant to be interpreted by
blink and this change is violating it.

I agree that current structure for WebURLError, ResourceError and net error is
more convoluted than necessary and we should probably fix it, but I'd prefer
doing that in a separate, dedicated one-off change.

[Shao-Chuan Lee, 2016/10/25 04:18:20]

I feel like using an enum for mapping from net:: errors we care about should be
better, the existing isCancellation field can also be replaced. ResourceError
will also have to be modified to be consistent.

Adding a new field is a rather simple solution though. kouhei@ WDYT?

+  return error;
+}
+
+bool WebURLError::isCacheMiss() const {
+  return NetworkUtils::isCacheMissErrorCode(reason);
+}
+
 }  // namespace blink