Send navigation_start to the browser in DidStartProvisionalLoad IPC

content/browser/frame_host/navigator.h

 // Copyright 2013 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_BROWSER_FRAME_HOST_NAVIGATOR_H_
 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/time/time.h"
 #include "content/browser/frame_host/navigator_delegate.h"
@@ skipping 35 matching lines @@
  public:
   // Returns the delegate of this Navigator.
   virtual NavigatorDelegate* GetDelegate();
 
   // Returns the NavigationController associated with this Navigator.
   virtual NavigationController* GetController();
 
   // Notifications coming from the RenderFrameHosts ----------------------------
 
   // The RenderFrameHostImpl started a provisional load.
-  virtual void DidStartProvisionalLoad(RenderFrameHostImpl* render_frame_host,
-                                       const GURL& url) {};
+  virtual void DidStartProvisionalLoad(
+      RenderFrameHostImpl* render_frame_host,
+      const GURL& url,
+      const base::TimeTicks& navigation_start){};

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

nit: There should be space between ) and {}. Is this clang-format clean?

[Charlie Harrison, 2015/11/14 00:25:00]

That's odd. clang-format actually prefers no space for me. Added the space.

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

This might be some bug in clang-format. If it repros for you, file a bug. It
seems inconsistent with the rest of the file, so let's leave it with the space
in and investigate the possible bug async.

 
   // The RenderFrameHostImpl has failed a provisional load.
   virtual void DidFailProvisionalLoadWithError(
       RenderFrameHostImpl* render_frame_host,
       const FrameHostMsg_DidFailProvisionalLoadWithError_Params& params) {};
 
   // The RenderFrameHostImpl has failed to load the document.
   virtual void DidFailLoadWithError(
       RenderFrameHostImpl* render_frame_host,
       const GURL& url,
@@ skipping 109 matching lines @@
       const base::TimeTicks& renderer_before_unload_end_time) {}
 
  protected:
   friend class base::RefCounted<Navigator>;
   virtual ~Navigator() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_