Support cross-process navigations in a single subframe RenderFrameHost.

content/browser/frame_host/navigation_controller_impl.h

Index: content/browser/frame_host/navigation_controller_impl.h
diff --git a/content/browser/frame_host/navigation_controller_impl.h b/content/browser/frame_host/navigation_controller_impl.h
index 99fc36ae00209adf8958624d920b82ce251c5d6c..e9206b4b72430269b023cc1017832b0c51c7e43b 100644
--- a/content/browser/frame_host/navigation_controller_impl.h
+++ b/content/browser/frame_host/navigation_controller_impl.h
@@ -134,7 +134,10 @@ class CONTENT_EXPORT NavigationControllerImpl
   //
   // In the case that nothing has changed, the details structure is undefined
   // and it will return false.
-  bool RendererDidNavigate(const ViewHostMsg_FrameNavigate_Params& params,
+  //
+  // TODO(creis): Change RenderViewHost to RenderFrameHost.
+  bool RendererDidNavigate(RenderViewHost* rvh,

[Charlie Reis, 2013/12/20 23:20:08]

If you want, these changes to NavigationController and
NavigationControllerDelegate can be carved out into their own CL, separate from
the --site-per-process logic.  Ok either way.

+                           const ViewHostMsg_FrameNavigate_Params& params,
                            LoadCommittedDetails* details);
 
   // Notifies us that we just became active. This is used by the WebContentsImpl
@@ -231,6 +234,7 @@ class CONTENT_EXPORT NavigationControllerImpl
 
   // Classifies the given renderer navigation (see the NavigationType enum).
   NavigationType ClassifyNavigation(
+      RenderViewHost* rvh,
       const ViewHostMsg_FrameNavigate_Params& params) const;
 
   // Causes the controller to load the specified entry. The function assumes
@@ -250,17 +254,27 @@ class CONTENT_EXPORT NavigationControllerImpl
   // The functions taking |did_replace_entry| will fill into the given variable
   // whether the last entry has been replaced or not.
   // See LoadCommittedDetails.did_replace_entry.
+  //
+  // TODO(creis): Change RenderViewHost to RenderFrameHost.
   void RendererDidNavigateToNewPage(
-      const ViewHostMsg_FrameNavigate_Params& params, bool replace_entry);
+      RenderViewHost* rvh,
+      const ViewHostMsg_FrameNavigate_Params& params,
+      bool replace_entry);
   void RendererDidNavigateToExistingPage(
+      RenderViewHost* rvh,
       const ViewHostMsg_FrameNavigate_Params& params);
   void RendererDidNavigateToSamePage(
+      RenderViewHost* rvh,
       const ViewHostMsg_FrameNavigate_Params& params);
   void RendererDidNavigateInPage(
-      const ViewHostMsg_FrameNavigate_Params& params, bool* did_replace_entry);
+      RenderViewHost* rvh,
+      const ViewHostMsg_FrameNavigate_Params& params,
+      bool* did_replace_entry);
   void RendererDidNavigateNewSubframe(
+      RenderViewHost* rvh,
       const ViewHostMsg_FrameNavigate_Params& params);
   bool RendererDidNavigateAutoSubframe(
+      RenderViewHost* rvh,
       const ViewHostMsg_FrameNavigate_Params& params);
 
   // Helper function for code shared between Reload() and ReloadIgnoringCache().