Send navigation_start to the browser in DidStartProvisionalLoad IPC

content/browser/frame_host/navigator.h

Index: content/browser/frame_host/navigator.h
diff --git a/content/browser/frame_host/navigator.h b/content/browser/frame_host/navigator.h
index e2f737d586ff0a21738c60ff6a2f7213b9e86166..2593aaaaa04f6e3e81997fb172a698b52c6f201d 100644
--- a/content/browser/frame_host/navigator.h
+++ b/content/browser/frame_host/navigator.h
@@ -53,8 +53,10 @@ class CONTENT_EXPORT Navigator : public base::RefCounted<Navigator> {
   // 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(