Send navigation_start to the browser in DidStartProvisionalLoad IPC

content/browser/frame_host/navigation_handle_impl.h

Index: content/browser/frame_host/navigation_handle_impl.h
diff --git a/content/browser/frame_host/navigation_handle_impl.h b/content/browser/frame_host/navigation_handle_impl.h
index 6f46707108e531ceac132639638d387611e5710f..2bb31ed5978ef05c85af55b6eb9031205fbe14c9 100644
--- a/content/browser/frame_host/navigation_handle_impl.h
+++ b/content/browser/frame_host/navigation_handle_impl.h
@@ -59,7 +59,8 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
  public:
   static scoped_ptr<NavigationHandleImpl> Create(
       const GURL& url,
-      FrameTreeNode* frame_tree_node);
+      FrameTreeNode* frame_tree_node,
+      const base::TimeTicks& navigation_start);
   ~NavigationHandleImpl() override;
 
   // NavigationHandle implementation:
@@ -90,6 +91,9 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
       const GURL& new_referrer_url,
       bool new_is_external_protocol) override;
 
+  // TODO(csharrison) add this to the public class and mark this override.

[clamy, 2015/11/06 16:28:20]

nit: please put the comment that was with the public method here (it can go back
to the public .h file when the method is made public).

+  const base::TimeTicks& GetNavigationStart();
+
   NavigatorDelegate* GetDelegate() const;
 
   void set_net_error_code(net::Error net_error_code) {
@@ -161,8 +165,13 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
     DID_COMMIT_ERROR_PAGE,
   };
 
+  // |navigation_start| comes from the DidStartProvisionalLoad IPC, which tracks

[clamy, 2015/11/06 16:28:20]

nit: I would move this comment to the factory method in the public part.

+  // both renderer-initiated and browser-initiated navigation start.
+  // PlzNavigate: This value always comes from the CommonNavigationParams
+  // associated with this navigation.
   NavigationHandleImpl(const GURL& url,
-                       FrameTreeNode* frame_tree_node);
+                       FrameTreeNode* frame_tree_node,
+                       const base::TimeTicks& navigation_start);
 
   NavigationThrottle::ThrottleCheckResult CheckWillStartRequest();
   NavigationThrottle::ThrottleCheckResult CheckWillRedirectRequest();
@@ -197,6 +206,9 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
   // The index of the next throttle to check.
   size_t next_index_;
 
+  // The time this navigation started.
+  const base::TimeTicks navigation_start_;
+
   // This callback will be run when all throttle checks have been performed.
   ThrottleChecksFinishedCallback complete_callback_;