Cache-aware Resource loading

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

Index: third_party/WebKit/Source/core/fetch/Resource.h
diff --git a/third_party/WebKit/Source/core/fetch/Resource.h b/third_party/WebKit/Source/core/fetch/Resource.h
index d531ba9238a701cf07b63fb27007c78c9902ddf8..dbd67115c3bebef87107b1f2f1fc0a001d5d8367 100644
--- a/third_party/WebKit/Source/core/fetch/Resource.h
+++ b/third_party/WebKit/Source/core/fetch/Resource.h
@@ -310,6 +310,17 @@ class CORE_EXPORT Resource : public GarbageCollectedFinalized<Resource>,
 
   virtual bool canReuse(const ResourceRequest&) const { return true; }
 
+  // If cache-aware loading is activated, this callback is called when the first
+  // disk-cache-only request failed due to cache miss.
+  virtual void willReloadAfterDiskCacheMiss() {}

[kinuko, 2016/11/08 06:10:18]

nit/optional:

This method naming feels a little weird to me, because 1) the willDoFoo() naming
pattern is often used to notify observers right before doFoo() is called while
we're not in that pattern in this change, and 2) the method name seems to imply
very specific action (i.e. reload) to be taken while the default impl does
nothing. 

I slightly prefer having a more generic name in the base class like:
didDetectCacheMiss()

and have a comment to note, say, 'subclasses (e.g. FontResource) may trigger
reloading to actually fetch the resource from network'.  Wdyt..?

[Shao-Chuan Lee, 2016/11/08 06:28:01]

SGTM, but I wonder if it's better to mention "disk" cache miss to distinguish
from memory cache miss? Naming is hard...

The reload is actually triggered by ResourceLoader, but it's indeed nice to have
comments here.

[kinuko, 2016/11/08 06:42:15]

Oh that's true, so maybe the current name is just ok.  Could we still slightly
improve the comment to note that reload is triggered right after this method by
ResourceLoader then?

[Shao-Chuan Lee, 2016/11/08 07:19:06]

Done, PTAL

+
+  // TODO(shaochuan): This is for saving back the actual ResourceRequest sent
+  // in ResourceFetcher::startLoad() for retry in cache-aware loading, remove
+  // once ResourceRequest is not modified in startLoad(). crbug.com/632580
+  void setResourceRequest(const ResourceRequest& resourceRequest) {
+    m_resourceRequest = resourceRequest;
+  }
+
   // Used by the MemoryCache to reduce the memory consumption of the entry.
   void prune();