Move out DidStartProvisionalLoad from WebContentsImpl into Navigator.

content/public/browser/render_frame_host.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_PUBLIC_BROWSER_RENDER_FRAME_HOST_H_
 #define CONTENT_PUBLIC_BROWSER_RENDER_FRAME_HOST_H_
 
 #include "content/common/content_export.h"
 #include "ipc/ipc_listener.h"
 #include "ipc/ipc_sender.h"
 
 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;
   RenderFrameHost() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_RENDER_FRAME_HOST_H_