FontLoadHistograms: classify as memory cache hit if FontResource loading not triggered

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 9bd6946fa80a57ec6af840e1424520f7fbcdeba2..36e203efd2b241c930717cb54ee01ea0ceb20456 100644
--- a/third_party/WebKit/Source/core/css/RemoteFontFaceSource.h
+++ b/third_party/WebKit/Source/core/css/RemoteFontFaceSource.h
@@ -74,7 +74,10 @@ private:
         void recordRemoteFont(const FontResource*);
         bool hadBlankText() { return m_blankPaintTime; }
         DataSource dataSource() { return m_dataSource; }
-        void maySetDataSource(DataSource dataSource) { m_dataSource = (m_dataSource != FromUnknown) ? m_dataSource : dataSource; }
+        // Classify as memory cache hit if |m_loadStartTime| is not set, i.e.
+        // this RemoteFontFaceSource instance didn't trigger FontResource
+        // loading.
+        void maySetDataSource(DataSource dataSource) { m_dataSource = (m_dataSource != FromUnknown) ? m_dataSource : (m_loadStartTime == 0) ? FromMemoryCache : dataSource; }

[kouhei (in TOK), 2016/09/30 01:18:35]

Please implement this in .cpp file, not nesting ?: op.

     private:
         void recordLoadTimeHistogram(const FontResource*, int duration);
         void recordInterventionResult(bool isTriggered);