WebFonts: measure network loading time

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 6bb1e5fc6f1c1bf59bd20ebfb55e6217406994ab..bfec96cb50f16a916e14f665a6797229f8357c3a 100644
--- a/third_party/WebKit/Source/core/css/RemoteFontFaceSource.h
+++ b/third_party/WebKit/Source/core/css/RemoteFontFaceSource.h
@@ -70,7 +70,7 @@ private:
         void recordRemoteFont(const FontResource*, bool isLoadedFromMemoryCache);
         bool hadBlankText() { return m_blankPaintTime; }
     private:
-        void recordLoadTimeHistogram(const FontResource*, int duration);
+        void recordLoadTimeHistogram(const FontResource*, int duration, bool isLoadedFromNetwork);

[kouhei (in TOK), 2016/09/02 03:22:20]

Can we have a enum for describing the source?
Looking at the below diff, we already have isLoadedFromMemoryCache which seems
to be opposite of this.

[Takashi Toyoshima, 2016/09/02 10:51:00]

Actually, it's not opposite of this since we have two other cases. But that
means enum is more preferable here.

Once concern is that the source couldn't be determined at a single timing, but
at multiple events. But I think we can manage it easily, and could reduce risks
to make mistakes.

         void recordInterventionResult(bool isTriggered, bool isLoadedFromNetwork);
         double m_loadStartTime;
         double m_blankPaintTime;