Add NavigationToFirstContentfulPaint UMA for ServiceWorker controlled pages.

components/page_load_metrics/renderer/metrics_render_frame_observer.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/page_load_metrics/renderer/metrics_render_frame_observer.h"
 
 #include <string>
 
 #include "base/time/time.h"
 #include "base/timer/timer.h"
@@ skipping 23 matching lines @@
     : content::RenderFrameObserver(render_frame) {}
 
 MetricsRenderFrameObserver::~MetricsRenderFrameObserver() {}
 
 void MetricsRenderFrameObserver::DidChangePerformanceTiming() {
   SendMetrics();
 }
 
 void MetricsRenderFrameObserver::DidObserveLoadingBehavior(
     blink::WebLoadingBehaviorFlag behavior) {
-  DCHECK(page_timing_metrics_sender_);
+  if (!page_timing_metrics_sender_)

[Bryan McQuade, 2016/04/14 15:10:26]

charles just landed a patch to fix this so you can omit it once you sync.

[horo, 2016/04/14 15:46:31]

I see. Synced.

+    return;
   page_timing_metrics_sender_->DidObserveLoadingBehavior(behavior);
 }
 
 void MetricsRenderFrameObserver::DidCommitProvisionalLoad(
     bool is_new_navigation,
     bool is_same_page_navigation) {
   // Same-page navigations (e.g. an in-document navigation from a fragment
   // link) aren't full page loads, since they don't go to network to load the
   // main HTML resource. DidStartProvisionalLoad doesn't get invoked for same
   // page navigations, so we may still have an active
@@ skipping 85 matching lines @@
   return make_scoped_ptr(new base::OneShotTimer);
 }
 
 bool MetricsRenderFrameObserver::HasNoRenderFrame() const {
   bool no_frame = !render_frame() || !render_frame()->GetWebFrame();
   DCHECK(!no_frame);
   return no_frame;
 }
 
 }  // namespace page_load_metrics