Move browser initiated navigation from RenderViewHost to RenderFrameHost.

content/browser/frame_host/render_frame_host_impl.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_RENDER_FRAME_HOST_IMPL_H_
 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
 
 #include <string>
 
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/strings/string16.h"
 #include "content/common/content_export.h"
 #include "content/public/browser/render_frame_host.h"
 
 class GURL;
 struct FrameHostMsg_DidFailProvisionalLoadWithError_Params;
+struct FrameMsg_Navigate_Params;
 
 namespace base {
 class FilePath;
 }
 
 namespace content {
 
 class CrossProcessFrameConnector;
 class FrameTree;
 class FrameTreeNode;
@@ skipping 58 matching lines @@
   }
 
   // Sets the RVH for |this| as pending shutdown. |on_swap_out| will be called
   // when the SwapOutACK is received.
   void SetPendingShutdown(const base::Closure& on_swap_out);
 
   // TODO(nasko): This method is public so RenderViewHostImpl::Navigate can
   // call it directly. It should be made private once Navigate moves here.
   void OnDidStartLoading();
 
+  // Sends the given navigation message. Use this rather than sending it
+  // yourself since this does the internal bookkeeping described below. This
+  // function takes ownership of the provided message pointer.
+  //
+  // If a cross-site request is in progress, we may be suspended while waiting
+  // for the onbeforeunload handler, so this function might buffer the message
+  // rather than sending it.
+  void Navigate(const FrameMsg_Navigate_Params& params);
+
+  // Load the specified URL, this is a shortcut for Navigate().

[Charlie Reis, 2014/02/13 01:23:53]

nit: As long as we're moving it, let's change the comma to a semicolon to fix
the run-on sentence.

[nasko, 2014/02/13 17:05:27]

Done.

+  void NavigateToURL(const GURL& url);
+
  protected:
   friend class RenderFrameHostFactory;
 
   // TODO(nasko): Remove dependency on RenderViewHost here. RenderProcessHost
   // should be the abstraction needed here, but we need RenderViewHost to pass
   // into WebContentsObserver::FrameDetached for now.
   RenderFrameHostImpl(RenderViewHostImpl* render_view_host,
                       RenderFrameHostDelegate* delegate,
                       FrameTree* frame_tree,
                       FrameTreeNode* frame_tree_node,
@@ skipping 63 matching lines @@
 
   int routing_id_;
   bool is_swapped_out_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_