[WIP] WebFonts cache-aware timeout adaption

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

 /*
  * Copyright (C) 2006, 2007, 2010, 2011 Apple Inc. All rights reserved.
  *           (C) 2007 Graham Dennis (graham.dennis@gmail.com)
  *
  * 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.
@@ skipping 119 matching lines @@
     didFail(nullptr, ResourceError::cancelledDueToAccessCheckError(newURL));
 }
 
 bool ResourceLoader::willFollowRedirect(
     WebURLLoader*,
     WebURLRequest& passedNewRequest,
     const WebURLResponse& passedRedirectResponse) {
   DCHECK(!passedNewRequest.isNull());
   DCHECK(!passedRedirectResponse.isNull());
 
+  if (m_resource->resourceRequest().isCacheAwareLoadingActivated()) {
+    // Fail as disk cache miss and retry in ResourceFetcher::didFailLoading().
+    WebURLError error;
+    error.reason = 1;  // non-zero

[kouhei (in TOK), 2016/10/20 10:43:07]

Would it be possible to do "error.setIsCacheMiss()"?

[Shao-Chuan Lee, 2016/10/21 04:35:02]

This WebURLError object will be converted into ResourceError in
|m_fetcher->didFailLoading()| in didFail(). Here we set reason to non-zero to
bypass
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/platform/expor...
Blink does not interpret the meaning of reason, as suggested in
https://cs.chromium.org/chromium/src/third_party/WebKit/public/platform/WebUR...

+    error.isCacheMiss = true;
+    didFail(nullptr, error);
+    return false;
+  }
+
   ResourceRequest& newRequest(passedNewRequest.toMutableResourceRequest());
   const ResourceResponse& redirectResponse(
       passedRedirectResponse.toResourceResponse());
   newRequest.setRedirectStatus(
       ResourceRequest::RedirectStatus::FollowedRedirect);
 
   const KURL originalURL = newRequest.url();
 
   if (!m_fetcher->willFollowRedirect(m_resource.get(), newRequest,
                                      redirectResponse)) {
@@ skipping 109 matching lines @@
   // a 304, where it will overwrite the cached data we should be reusing.
   if (dataOut.size()) {
     m_fetcher->didReceiveData(m_resource.get(), dataOut.data(), dataOut.size(),
                               encodedDataLength);
     m_resource->setResourceBuffer(dataOut);
   }
   didFinishLoading(0, monotonicallyIncreasingTime(), encodedDataLength);
 }
 
 }  // namespace blink