[WIP] WebFonts cache-aware timeout adaption

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

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller <mueller@kde.org>
     Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
     Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
     version 2 of the License, or (at your option) any later version.
@@ skipping 17 matching lines @@
 #include "core/fetch/ResourceLoaderOptions.h"
 #include "platform/MemoryCoordinator.h"
 #include "platform/SharedBuffer.h"
 #include "platform/Timer.h"
 #include "platform/network/ResourceError.h"
 #include "platform/network/ResourceLoadPriority.h"
 #include "platform/network/ResourceRequest.h"
 #include "platform/network/ResourceResponse.h"
 #include "platform/scheduler/CancellableTaskFactory.h"
 #include "platform/web_process_memory_dump.h"
+#include "public/platform/WebCachePolicy.h"
 #include "public/platform/WebDataConsumerHandle.h"
 #include "wtf/Allocator.h"
 #include "wtf/HashCountedSet.h"
 #include "wtf/HashSet.h"
 #include "wtf/text/WTFString.h"
 #include <memory>
 
 namespace blink {
 
 struct FetchInitiatorInfo;
@@ skipping 231 matching lines @@
 
   void addToEncodedBodyLength(int value) {
     m_response.addToEncodedBodyLength(value);
   }
   void addToDecodedBodyLength(int value) {
     m_response.addToDecodedBodyLength(value);
   }
 
   virtual bool canReuse(const ResourceRequest&) const { return true; }
 
+  virtual void willReloadAfterDiskCacheMiss();

[yhirano, 2016/10/04 05:27:10]

Please add a comment that add (/ remove?)-ing clients is forbidden in this
function.

[Shao-Chuan Lee, 2016/10/04 09:08:04]

Done.

+
+  // TODO(632580): Workaround to persist cache-aware state, remove after fixed.

[kouhei (in TOK), 2016/10/03 08:11:47]

hiroshige-san: Would you take a look?

[hiroshige, 2016/10/03 08:28:20]

Nate is working on the issue:
https://codereview.chromium.org/2203613003/

+  void setResourceRequest(const ResourceRequest& resourceRequest) {
+    m_resourceRequest = resourceRequest;
+  }
+
   // Used by the MemoryCache to reduce the memory consumption of the entry.
   void prune();
 
   virtual void onMemoryDump(WebMemoryDumpLevelOfDetail,
                             WebProcessMemoryDump*) const;
 
   static const char* resourceTypeToString(Type, const FetchInitiatorInfo&);
 
  protected:
   Resource(const ResourceRequest&, Type, const ResourceLoaderOptions&);
@@ skipping 103 matching lines @@
 
   unsigned m_preloadResult : 2;  // PreloadResult
   unsigned m_type : 4;           // Type
   unsigned m_status : 3;         // Status
 
   unsigned m_needsSynchronousCacheHit : 1;
   unsigned m_linkPreload : 1;
   bool m_isRevalidating : 1;
   bool m_isAlive : 1;
 
+  bool m_isAddClientProhibited : 1;

[yhirano, 2016/10/04 05:27:10]

Is it a good idea to prohibit removeClient as well?

[Shao-Chuan Lee, 2016/10/04 09:08:04]

Done.

+
   // Ordered list of all redirects followed while fetching this resource.
   Vector<RedirectPair> m_redirectChain;
 
   HeapHashCountedSet<WeakMember<ResourceClient>> m_clients;
   HeapHashCountedSet<WeakMember<ResourceClient>> m_clientsAwaitingCallback;
   HeapHashCountedSet<WeakMember<ResourceClient>> m_finishedClients;
 
   ResourceLoaderOptions m_options;
 
   double m_responseTimestamp;
@@ skipping 23 matching lines @@
 };
 
 #define DEFINE_RESOURCE_TYPE_CASTS(typeName)                   \
   DEFINE_TYPE_CASTS(typeName##Resource, Resource, resource,    \
                     resource->getType() == Resource::typeName, \
                     resource.getType() == Resource::typeName);
 
 }  // namespace blink
 
 #endif