WebFonts intervention: Add metrics only for miss-cached loading

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 d52613625046933dc47470d147d0865a2e271af1..bfa962220c5162afd81144ecfed87e50c5dd52b7 100644
--- a/third_party/WebKit/Source/core/css/RemoteFontFaceSource.h
+++ b/third_party/WebKit/Source/core/css/RemoteFontFaceSource.h
@@ -64,14 +64,14 @@ private:
         FontLoadHistograms() : m_loadStartTime(0), m_blankPaintTime(0), m_isLongLimitExceeded(false) { }
         void loadStarted();
         void fallbackFontPainted(DisplayPeriod);
-        void fontLoaded(bool isInterventionTriggered);
+        void fontLoaded(bool isInterventionTriggered, bool isLoadedFromCache);

[kinuko, 2016/06/27 12:06:24]

usually prefer enum over bool... but ok this is internal histogram class

         void longLimitExceeded(bool isInterventionTriggered);
         void recordFallbackTime(const FontResource*);
         void recordRemoteFont(const FontResource*);
         bool hadBlankText() { return m_blankPaintTime; }
     private:
         void recordLoadTimeHistogram(const FontResource*, int duration);
-        void recordInterventionResult(bool triggered);
+        void recordInterventionResult(bool triggered, bool loadedFromCache);

[kinuko, 2016/06/27 12:06:24]

loadedFromCache -> isLoadedFromCache for consistency?

[Takashi Toyoshima, 2016/06/28 05:38:18]

Done with triggered -> isTriggered.

         double m_loadStartTime;
         double m_blankPaintTime;
         bool m_isLongLimitExceeded;
@@ -86,6 +86,7 @@ private:
     DisplayPeriod m_period;
     FontLoadHistograms m_histograms;
     bool m_isInterventionTriggered;
+    bool m_isLoadedFromMemoryCache;
 };
 
 } // namespace blink