Adding DRP specfic UMA for FirstContentfulPaint

content/public/browser/navigation_handle.h

Index: content/public/browser/navigation_handle.h
diff --git a/content/public/browser/navigation_handle.h b/content/public/browser/navigation_handle.h
index 124ef31b0ab042f9db51b1a4a4257c009f8a9dff..85298d0bae679ce56719fbb27c07feae2cc5e68c 100644
--- a/content/public/browser/navigation_handle.h
+++ b/content/public/browser/navigation_handle.h
@@ -9,20 +9,21 @@
 
 #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.
@@ -161,15 +162,20 @@ class CONTENT_EXPORT NavigationHandle {
                                  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_