Send navigation_start to the browser in DidStartProvisionalLoad IPC

content/browser/frame_host/navigation_handle_impl.cc

Index: content/browser/frame_host/navigation_handle_impl.cc
diff --git a/content/browser/frame_host/navigation_handle_impl.cc b/content/browser/frame_host/navigation_handle_impl.cc
index b1aace3597c5ecd773339efa3b2472beac3db8a7..204682edb9a92918292c49495d5367eb79b6d101 100644
--- a/content/browser/frame_host/navigation_handle_impl.cc
+++ b/content/browser/frame_host/navigation_handle_impl.cc
@@ -33,13 +33,16 @@ void UpdateThrottleCheckResult(
 // static
 scoped_ptr<NavigationHandleImpl> NavigationHandleImpl::Create(
     const GURL& url,
-    FrameTreeNode* frame_tree_node) {
+    FrameTreeNode* frame_tree_node,
+    const base::TimeTicks& navigation_start) {
   return scoped_ptr<NavigationHandleImpl>(
-      new NavigationHandleImpl(url, frame_tree_node));
+      new NavigationHandleImpl(url, frame_tree_node, navigation_start));
 }
 
-NavigationHandleImpl::NavigationHandleImpl(const GURL& url,
-                                           FrameTreeNode* frame_tree_node)
+NavigationHandleImpl::NavigationHandleImpl(
+    const GURL& url,
+    FrameTreeNode* frame_tree_node,
+    const base::TimeTicks& navigation_start)
     : url_(url),
       is_post_(false),
       has_user_gesture_(false),
@@ -51,7 +54,9 @@ NavigationHandleImpl::NavigationHandleImpl(const GURL& url,
       state_(INITIAL),
       is_transferring_(false),
       frame_tree_node_(frame_tree_node),
-      next_index_(0) {
+      next_index_(0),
+      navigation_start_(navigation_start) {
+  DCHECK(!navigation_start.is_null());
   // PlzNavigate
   // Initialize the ServiceWorkerNavigationHandle if it can be created for this
   // frame.
@@ -94,6 +99,10 @@ bool NavigationHandleImpl::IsInMainFrame() {
   return frame_tree_node_->IsMainFrame();
 }
 
+const base::TimeTicks& NavigationHandleImpl::GetNavigationStart() {

[nasko, 2015/11/13 21:52:54]

This is just an accessor, should be hacker_cased and inlined in the header file.

[Charlie Harrison, 2015/11/14 00:24:59]

Ah okay. Wasn't aware of that rule for accessors. Done.

[nasko, 2015/11/14 00:33:24]

Maybe freshen up on the style guide ;).

+  return navigation_start_;
+}
+
 bool NavigationHandleImpl::IsPost() {
   CHECK_NE(INITIAL, state_)
       << "This accessor should not be called before the request is started.";