(DEPRECATED) Send navigation_start to browser process in DidStartProvisionalLoad

content/common/navigation_params.h

Index: content/common/navigation_params.h
diff --git a/content/common/navigation_params.h b/content/common/navigation_params.h
index cf3d89a2cbb2fb9473b23918cecb02e030a3d161..566e3ddb7f7add6f9d9d50de1caaa46a1403b061 100644
--- a/content/common/navigation_params.h
+++ b/content/common/navigation_params.h
@@ -60,7 +60,8 @@ struct CONTENT_EXPORT CommonNavigationParams {
                          FrameMsg_UILoadMetricsReportType::Value report_type,
                          const GURL& base_url_for_data_url,
                          const GURL& history_url_for_data_url,
-                         LoFiState lofi_state);
+                         LoFiState lofi_state,
+                         const base::TimeTicks& navigation_start);

[clamy, 2015/11/02 17:34:35]

In which cases are we creating CommonNavigationParams with a navigation_start
value other than base::TimeTicks::Now()? If there are none, let's remove this
argument from the constructor.

[Charlie Harrison, 2015/11/02 18:19:10]

A few places:
- ConstructCommonNavigationParams uses it, which is used to initiate nav_start
in NavigatorImpl::NavigateToEntry for both RequestNavigation() and
render_frame_host->Navigate.
- In RenderFrameImpl::BeginNavigation we want to set the nav start timestamp
before we MakeCommonNavigationParams (this one could be argued)

   ~CommonNavigationParams();
 
   // The URL to navigate to.
@@ -107,6 +108,12 @@ struct CONTENT_EXPORT CommonNavigationParams {
   // Whether or not to request a LoFi version of the document or let the browser
   // decide.
   LoFiState lofi_state;
+
+  // The navigationStart time exposed through the Navigation Timing API to JS.
+  // If this is a browser-initiated navigation, this will override the
+  // navigation_start value in Blink. If this is a renderer-initiated
+  // navigation, this will be recorded in the renderer.
+  base::TimeTicks navigation_start;
 };
 
 // Provided by the renderer ----------------------------------------------------
@@ -204,7 +211,6 @@ struct CONTENT_EXPORT StartNavigationParams {
 struct CONTENT_EXPORT RequestNavigationParams {
   RequestNavigationParams();
   RequestNavigationParams(bool is_overriding_user_agent,
-                          base::TimeTicks navigation_start,
                           const std::vector<GURL>& redirects,
                           bool can_load_local_resources,
                           base::Time request_time,
@@ -223,9 +229,6 @@ struct CONTENT_EXPORT RequestNavigationParams {
   // Whether or not the user agent override string should be used.
   bool is_overriding_user_agent;
 
-  // The navigationStart time to expose through the Navigation Timing API to JS.
-  base::TimeTicks browser_navigation_start;
-
   // Any redirect URLs that occurred before |url|. Useful for cross-process
   // navigations; defaults to empty.
   std::vector<GURL> redirects;