FirstMeaningfulPaint UMA

chrome/renderer/page_load_metrics/metrics_render_frame_observer.cc

Index: chrome/renderer/page_load_metrics/metrics_render_frame_observer.cc
diff --git a/chrome/renderer/page_load_metrics/metrics_render_frame_observer.cc b/chrome/renderer/page_load_metrics/metrics_render_frame_observer.cc
index ef32d84b88bcfd6a76e543df7748571467afc67e..54fb495457c9c18e02d0b0f3f209df276d04f4a4 100644
--- a/chrome/renderer/page_load_metrics/metrics_render_frame_observer.cc
+++ b/chrome/renderer/page_load_metrics/metrics_render_frame_observer.cc
@@ -140,6 +140,9 @@ PageLoadTiming MetricsRenderFrameObserver::GetTiming() const {
   if (perf.firstContentfulPaint() > 0.0)
     timing.first_contentful_paint =
         ClampDelta(perf.firstContentfulPaint(), start);
+  if (perf.firstMeaningfulPaint() > 0.0)

[Charlie Harrison, 2016/08/04 02:18:29]

nit: Can you wrap the if with braces? I'm not a fan of multi-line if bodies
without braces. (Can do it for FCP too).

[Kunihiko Sakamoto, 2016/08/04 02:56:14]

Done.

+    timing.first_meaningful_paint =
+        ClampDelta(perf.firstMeaningfulPaint(), start);
   if (perf.parseStart() > 0.0)
     timing.parse_start = ClampDelta(perf.parseStart(), start);
   if (perf.parseStop() > 0.0)