PlzNavigate: transmit redirect info to the renderer side

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 997805938531dd69285fc32763d94d83c67f1a6c..77ec1bbc8dc7dfa44ae1d43cc5e864dab3ef8e0e 100644
--- a/content/browser/frame_host/navigator_impl.cc
+++ b/content/browser/frame_host/navigator_impl.cc
@@ -461,7 +461,8 @@ bool NavigatorImpl::NavigateToEntry(
               previews_state, navigation_start),
           entry.ConstructStartNavigationParams(),
           entry.ConstructRequestNavigationParams(
-              frame_entry, is_history_navigation_in_new_child,
+              frame_entry, GURL(), std::string(),

[nasko, 2017/02/10 23:53:09]

Shouldn't this be dest_url and the actual method?

[clamy, 2017/02/13 14:29:37]

I marked this parameter as PlzNavigate only, hence why I'm not initializing it
in the non-PlzNavigate case. In particular, I was not sure what to put in the
transfer navigation case: should this be the original url, or the url the
transferred renderer needs to navigate to? Since this is not meant to be used
outside of PlzNavigate, I though it was safer not to initialize it.

[nasko, 2017/02/14 17:25:02]

Acknowledged.

+              is_history_navigation_in_new_child,
               entry.GetSubframeUniqueNames(frame_tree_node),
               frame_tree_node->has_committed_real_load(),
               controller_->GetPendingEntryIndex() == -1,