Refine FontLoadHistograms::recordRemoteFont() logic

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

Index: third_party/WebKit/Source/core/css/RemoteFontFaceSource.cpp
diff --git a/third_party/WebKit/Source/core/css/RemoteFontFaceSource.cpp b/third_party/WebKit/Source/core/css/RemoteFontFaceSource.cpp
index b90978fb69e96151c5bbbafa07d3eae875576297..0ab617696e54174447b5c11c7827069f084ed6f0 100644
--- a/third_party/WebKit/Source/core/css/RemoteFontFaceSource.cpp
+++ b/third_party/WebKit/Source/core/css/RemoteFontFaceSource.cpp
@@ -265,13 +265,12 @@ void RemoteFontFaceSource::FontLoadHistograms::recordFallbackTime(const FontReso
 
 void RemoteFontFaceSource::FontLoadHistograms::recordRemoteFont(const FontResource* font)
 {
-    if (m_loadStartTime > 0 && font && !font->isLoading()) {
-        DEFINE_STATIC_LOCAL(EnumerationHistogram, cacheHitHistogram, ("WebFont.CacheHit", CacheHitEnumMax));
-        cacheHitHistogram.count(dataSourceMetricsValue());
+    DEFINE_STATIC_LOCAL(EnumerationHistogram, cacheHitHistogram, ("WebFont.CacheHit", CacheHitEnumMax));
+    cacheHitHistogram.count(dataSourceMetricsValue());
 
+    if (m_dataSource == FromDiskCache || m_dataSource == FromNetwork) {
         int duration = static_cast<int>(currentTimeMS() - m_loadStartTime);
         recordLoadTimeHistogram(font, duration);
-        m_loadStartTime = -1;
 
         enum { CORSFail, CORSSuccess, CORSEnumMax };
         int corsValue = font->isCORSFailed() ? CORSFail : CORSSuccess;