Store NavigationEntryImpl data into NavigationRequest for later usage.

content/browser/frame_host/navigator_impl.cc

Index: content/browser/frame_host/navigator_impl.cc
diff --git a/content/browser/frame_host/navigator_impl.cc b/content/browser/frame_host/navigator_impl.cc
index c680cb199fecf72ebc900ce25a3088608063a288..d3fdb909f31d6086dd40ca0a63fd478b941349aa 100644
--- a/content/browser/frame_host/navigator_impl.cc
+++ b/content/browser/frame_host/navigator_impl.cc
@@ -783,7 +783,7 @@ void NavigatorImpl::OnBeginNavigation(
 
   // Then notify the RenderFrameHostManager so it can speculatively create a
   // RenderFrameHost (and potentially a new renderer process) in parallel.
-  frame_tree_node->render_manager()->BeginNavigation(common_params);
+  frame_tree_node->render_manager()->BeginNavigation(navigation_request);
 }
 
 // PlzNavigate
@@ -809,8 +809,7 @@ void NavigatorImpl::CommitNavigation(FrameTreeNode* frame_tree_node,
   // Select an appropriate renderer to commit the navigation.
   RenderFrameHostImpl* render_frame_host =
       frame_tree_node->render_manager()->GetFrameHostForNavigation(
-          navigation_request->common_params().url,
-          navigation_request->common_params().transition);
+          navigation_request);
   CheckWebUIRendererDoesNotDisplayNormalURL(
       render_frame_host, navigation_request->common_params().url);
 
@@ -895,6 +894,7 @@ bool NavigatorImpl::RequestNavigation(
       CommitNavigationParams(entry.GetPageState(),
                              entry.GetIsOverridingUserAgent(),
                              navigation_start)));
+  navigation_request->CopyDataFrom(entry);

[clamy, 2015/01/16 16:39:02]

Considering that we only call CopyDataFrom once, just below the constructor, how
about making NavigationEntry a parameter of the constructor and doing the work
there?

[carlosk, 2015/01/19 15:02:52]

Done.

   RequestNavigationParams request_params(entry.GetHasPostData(),
                                          entry.extra_headers(),
                                          entry.GetBrowserInitiatedPostData());