PlzNavigate: provide the FrameTreeNode ID to the network stack

content/browser/frame_host/frame_tree_node.cc

Index: content/browser/frame_host/frame_tree_node.cc
diff --git a/content/browser/frame_host/frame_tree_node.cc b/content/browser/frame_host/frame_tree_node.cc
index 15679805e80deb41c13d12ad670af4db0ea014df..7a1ed1ce731dd3b2bd4d77273e5cbd1778d6016c 100644
--- a/content/browser/frame_host/frame_tree_node.cc
+++ b/content/browser/frame_host/frame_tree_node.cc
@@ -180,6 +180,7 @@ void FrameTreeNode::SetNavigationRequest(
   CHECK(base::CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kEnableBrowserSideNavigation));
   ResetNavigationRequest(false);
+  DidStartLoading(true);

[nasko, 2015/04/27 14:24:11]

Does that and the DidStopLoading code fix issues on their own? I wonder if they
belong logically in a separate CL.

[clamy, 2015/04/27 14:34:44]

No this was a test patch that I uploaded and was reviewing before sending it
out. I have to add this locally for the DidStart/StopLoading to get signaled and
the tests to work (and normally remove it when sending patches out but forgot
here). Hopefully Fabrice's CL will be ready soon.

   // TODO(clamy): perform the StartLoading logic here.
   navigation_request_ = navigation_request.Pass();
 }
@@ -192,6 +193,7 @@ void FrameTreeNode::ResetNavigationRequest(bool is_commit) {
   // corresponds to a cancelation, the RenderFrameHostManager should clean up
   // any speculative RenderFrameHost it created for the navigation.
   if (navigation_request_ && !is_commit) {
+    DidStopLoading();
     // TODO(clamy): perform the StopLoading logic.
     render_manager_.CleanUpNavigation();
   }