Move out DidStartProvisionalLoad from WebContentsImpl into Navigator.

content/public/browser/render_frame_host.h

Index: content/public/browser/render_frame_host.h
diff --git a/content/public/browser/render_frame_host.h b/content/public/browser/render_frame_host.h
index 9b6344f3bae54ec65a6bffd35bc7e62a51d92871..6a563895fc2acc1c982ddeaab1210c6a1cafa917 100644
--- a/content/public/browser/render_frame_host.h
+++ b/content/public/browser/render_frame_host.h
@@ -11,12 +11,19 @@
 
 namespace content {
 
+class RenderProcessHost;
+class SiteInstance;
+
 // The interface provides a communication conduit with a frame in the renderer.
 class CONTENT_EXPORT RenderFrameHost : public IPC::Listener,
                                        public IPC::Sender {
  public:
   virtual ~RenderFrameHost() {}
 
+  virtual RenderProcessHost* GetProcess() const = 0;

[Charlie Reis, 2013/11/07 01:22:43]

Do we need this?  You can get the process from the SiteInstance.

Actually, I'm not entirely clear why we need to pass RenderFrameHosts to
Navigator.  Is Navigator going to become public, or could it just use
RenderFrameHostImpl?

+
+  virtual SiteInstance* GetSiteInstance() const = 0;
+
  private:
   // This interface should only be implemented inside content.
   friend class RenderFrameHostImpl;