Create RenderFrameProxyHost at time of swap-out instead of commit.

content/browser/renderer_host/render_view_host_impl.h

Index: content/browser/renderer_host/render_view_host_impl.h
diff --git a/content/browser/renderer_host/render_view_host_impl.h b/content/browser/renderer_host/render_view_host_impl.h
index 4e3d5ced4225495b3ff22b7064a1eddb2f792c57..3532e31f7afefad8369112cf47336e116df94c45 100644
--- a/content/browser/renderer_host/render_view_host_impl.h
+++ b/content/browser/renderer_host/render_view_host_impl.h
@@ -233,6 +233,7 @@ class CONTENT_EXPORT RenderViewHostImpl
   // created with an opener. (The opener may have been closed since.)
   virtual bool CreateRenderView(const base::string16& frame_name,
                                 int opener_route_id,
+                                int proxy_routing_id,

[Charlie Reis, 2014/05/15 00:32:50]

This feels strange to me.  A RenderFrameProxy is frame-specific, unlike
RenderView, and most RenderViews won't have them.  Is there another way to
handle this?  (We should at least have a comment about this similar to the one
in view_messages.h, but it would be nice to avoid passing it here if possible.)

minor nit: It looks a little strange to have route vs routing in the two
parameter names.

[nasko, 2014/05/15 18:47:13]

As discussed, there isn't any other way that will be much cleaner, so leaving it
in place. I've moved the proxy routing id to be next to swapped out state
wherever it is encountered and fixed the route vs routing here.

                                 int32 max_page_id,
                                 bool window_was_created_with_opener);