Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(223)

Unified Diff: third_party/WebKit/Source/core/fetch/FontResource.h

Issue 2390583002: [WIP] WebFonts cache-aware timeout adaption (Closed)
Patch Set: check ERR_CACHE_MISS directly in ResourceError Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/fetch/FontResource.h
diff --git a/third_party/WebKit/Source/core/fetch/FontResource.h b/third_party/WebKit/Source/core/fetch/FontResource.h
index e83ab315a525782642b34205c432c2d1cc84114f..c3676fe6d706f3dd1f4fa646bea5279e2d56b1ac 100644
--- a/third_party/WebKit/Source/core/fetch/FontResource.h
+++ b/third_party/WebKit/Source/core/fetch/FontResource.h
@@ -67,6 +67,10 @@ class CORE_EXPORT FontResource final : public Resource {
bool italic,
FontOrientation = FontOrientation::Horizontal);
+ // Calls to addClient() and removeClient() is forbidden in this callback to
+ // prevent potential race issues.
+ void willReloadAfterDiskCacheMiss() override;
+
private:
class FontResourceFactory : public ResourceFactory {
public:
@@ -112,6 +116,7 @@ class FontResourceClient : public ResourceClient {
ResourceClientType getResourceClientType() const final { return FontType; }
virtual void fontLoadShortLimitExceeded(FontResource*) {}
virtual void fontLoadLongLimitExceeded(FontResource*) {}
+ virtual void willReloadAfterDiskCacheMiss(const FontResource*) {}
};
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698