UMA histograms to measure whether / how long blank text is rendered for loading web fonts

Source/core/css/CSSFontFaceSource.cpp

Index: Source/core/css/CSSFontFaceSource.cpp
diff --git a/Source/core/css/CSSFontFaceSource.cpp b/Source/core/css/CSSFontFaceSource.cpp
index 62521dcb13e6822dfe634a2878b22dcb552ae913..da6241d914cbb6e8ac462b2b23d8bfdb6d0872c5 100644
--- a/Source/core/css/CSSFontFaceSource.cpp
+++ b/Source/core/css/CSSFontFaceSource.cpp
@@ -205,6 +205,8 @@ PassRefPtr<SimpleFontData> CSSFontFaceSource::getFontData(const FontDescription&
                 if (!m_font->ensureCustomFontData())
                     return nullptr;
 
+                m_histograms.recordFallbackTime(m_font.get());
+
                 fontData = SimpleFontData::create(
                     m_font->platformDataFromCustomData(fontDescription.effectiveFontSize(),
                         fontDescription.isSyntheticBold(), fontDescription.isSyntheticItalic(),
@@ -285,6 +287,12 @@ void CSSFontFaceSource::FontLoadHistograms::loadStarted()
         m_loadStartTime = currentTimeMS();
 }
 
+void CSSFontFaceSource::FontLoadHistograms::fallbackFontPainted()
+{
+    if (!m_fallbackPaintTime)
+        m_fallbackPaintTime = currentTimeMS();
+}
+
 void CSSFontFaceSource::FontLoadHistograms::recordLocalFont(bool loadSuccess)
 {
     if (!m_loadStartTime) {
@@ -293,6 +301,15 @@ void CSSFontFaceSource::FontLoadHistograms::recordLocalFont(bool loadSuccess)
     }
 }
 
+void CSSFontFaceSource::FontLoadHistograms::recordFallbackTime(const FontResource* font)
+{
+    if (m_fallbackPaintTime > 0) {

[dglazkov, 2014/03/04 17:24:08]

Early return?

[Kunihiko Sakamoto, 2014/03/05 08:55:59]

Done.

+        int duration = static_cast<int>(currentTimeMS() - m_fallbackPaintTime);
+        blink::Platform::current()->histogramCustomCounts("WebFont.BlankTextShownTime", duration, 0, 10000, 50);
+        m_fallbackPaintTime = -1;
+    }
+}
+
 void CSSFontFaceSource::FontLoadHistograms::recordRemoteFont(const FontResource* font)
 {
     if (m_loadStartTime > 0 && font && !font->isLoading()) {