RemoteFontFaceSource should keep FontCustomPlatformData over FontResource revalidation

third_party/WebKit/Source/core/css/RemoteFontFaceSource.h

Index: third_party/WebKit/Source/core/css/RemoteFontFaceSource.h
diff --git a/third_party/WebKit/Source/core/css/RemoteFontFaceSource.h b/third_party/WebKit/Source/core/css/RemoteFontFaceSource.h
index beb46b50ff2b0a8ee826b47501fe6314f3699819..1fb145d920dce793d50ca2e423df53e105de592d 100644
--- a/third_party/WebKit/Source/core/css/RemoteFontFaceSource.h
+++ b/third_party/WebKit/Source/core/css/RemoteFontFaceSource.h
@@ -12,6 +12,7 @@
 namespace blink {
 
 class CSSFontSelector;
+class FontCustomPlatformData;
 
 enum FontDisplay {
   FontDisplayAuto,
@@ -88,7 +89,7 @@ class RemoteFontFaceSource final : public CSSFontFaceSource,
                     bool loadError,
                     bool isInterventionTriggered);
     void longLimitExceeded(bool isInterventionTriggered);
-    void recordFallbackTime(const FontResource*);
+    void recordFallbackTime();
     void recordRemoteFont(const FontResource*, bool isInterventionTriggered);
     bool hadBlankText() { return m_blankPaintTime; }
     DataSource dataSource() { return m_dataSource; }
@@ -114,6 +115,7 @@ class RemoteFontFaceSource final : public CSSFontFaceSource,
 
   Member<FontResource> m_font;

[Takashi Toyoshima, 2017/02/27 12:56:02]

Can we start having comments on members?
Probably, it would be better to say m_font will be reset once loading is
finished. That explains your isLoaded() implementation in this class.

[Kunihiko Sakamoto, 2017/02/28 04:04:02]

Done.

   Member<CSSFontSelector> m_fontSelector;
+  RefPtr<FontCustomPlatformData> m_customFontData;
   const FontDisplay m_display;
   DisplayPeriod m_period;
   FontLoadHistograms m_histograms;