Adding DRP specfic UMA for FirstContentfulPaint

content/public/browser/navigation_handle.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_
 #define CONTENT_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_
 
 #include <memory>
 
 #include "content/common/content_export.h"
 #include "content/public/browser/navigation_throttle.h"
 #include "content/public/common/referrer.h"
 #include "net/base/net_errors.h"
 #include "ui/base/page_transition_types.h"
 
 class GURL;
 
 namespace content {
+class NavigationData;
 class NavigationThrottle;
 class RenderFrameHost;
 class WebContents;
 
 // A NavigationHandle tracks information related to a single navigation.
 // NavigationHandles are provided to several WebContentsObserver methods to
 // allow observers to track specific navigations. Observers should clear any
 // references to a NavigationHandle at the time of
 // WebContentsObserver::DidFinishNavigation, just before the handle is
 // destroyed.
@@ skipping 132 matching lines @@
                                  bool has_user_gesture,
                                  ui::PageTransition transition,
                                  bool is_external_protocol) = 0;
 
   // Simulates the network request being redirected.
   virtual NavigationThrottle::ThrottleCheckResult
   CallWillRedirectRequestForTesting(const GURL& new_url,
                                     bool new_method_is_post,
                                     const GURL& new_referrer_url,
                                     bool new_is_external_protocol) = 0;
+
+  // The NavigationData that the embedder returned from
+  // ResourceDispatcherHostDelegate::GetNavigationData during commit. This will
+  // be a clone of the NavigationData.
+  virtual const NavigationData* GetNavigationData() const = 0;

[nasko, 2016/05/09 17:35:43]

I think we already covered that there is no const usage in the content/public/
API - https://www.chromium.org/developers/content-module/content-api.

[RyanSturm, 2016/05/09 18:27:31]

Done. Removed both consts as it seems like the embedder should be allowed to do
whatever it wants with the NavigationData, and shouldn't be constrained to keep
it the same.

 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_