Move out DidStartProvisionalLoad from WebContentsImpl into Navigator.

content/browser/frame_host/frame_tree_node.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_FRAME_TREE_NODE_H_
 #define CONTENT_BROWSER_FRAME_HOST_FRAME_TREE_NODE_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 15 matching lines @@
 // are frame-specific (as opposed to page-specific).
 class CONTENT_EXPORT FrameTreeNode {
  public:
   static const int64 kInvalidFrameId;
 
   FrameTreeNode(Navigator* navigator,
                 RenderViewHostDelegate* render_view_delegate,
                 RenderWidgetHostDelegate* render_widget_delegate,
                 RenderViewHostManager::Delegate* manager_delegate,
                 int64 frame_id,
-                const std::string& name,
-                scoped_ptr<RenderFrameHostImpl> render_frame_host);
+                const std::string& name);
 
   ~FrameTreeNode();
 
   void AddChild(scoped_ptr<FrameTreeNode> child);
   void RemoveChild(FrameTreeNode* child);
 
+  // TODO(nasko): This method should be removed once there is a
+  // RenderFrameHostManager for each node in the frame tree.
+  void set_render_frame_host(

[Charlie Reis, 2013/11/21 21:59:32]

I'm a bit uncomfortable about decoupling this from owns_render_frame_host_.  Do
we need this method?

[nasko, 2013/11/22 01:02:34]

Yes, we do. Since RFH is created after FTN, until we have RFHM we need to hand
the RFH to FTN. The code before this CL got away with RFH being created before
FTN.

+      scoped_ptr<RenderFrameHostImpl> render_frame_host) {
+    render_frame_host_ = render_frame_host.release();
+  }
+
   // Transitional API allowing the RenderFrameHost of a FrameTreeNode
   // representing the main frame to be provided by someone else. After
   // this is called, the FrameTreeNode no longer owns its RenderFrameHost.
   //
   // This should only be used for the main frame (aka root) in a frame tree.
   //
   // TODO(ajwong): Remove this method once the main frame RenderFrameHostImpl is
   // no longer owned by the RenderViewHostImpl.
   void ResetForMainFrame(RenderFrameHostImpl* new_render_frame_host);
 
@@ skipping 97 matching lines @@
   // TODO(creis): Remove this when we can store subframe URLs in the
   // NavigationController.
   GURL current_url_;
 
   DISALLOW_COPY_AND_ASSIGN(FrameTreeNode);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_FRAME_TREE_NODE_H_