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

content/common/frame_messages.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.
 
 // IPC messages for interacting with frames.
 // Multiply-included message file, hence no include guard.
 
 #include "content/common/content_export.h"
 #include "content/common/content_param_traits.h"
 #include "content/common/frame_message_enums.h"
@@ skipping 283 matching lines @@
 
 // Tells the renderer to perform the specified navigation, interrupting any
 // existing navigation.
 IPC_MESSAGE_ROUTED1(FrameMsg_Navigate, FrameMsg_Navigate_Params)
 
 // Instructs the renderer to invoke the frame's beforeunload event handler.
 // Expects the result to be returned via FrameHostMsg_BeforeUnload_ACK.
 IPC_MESSAGE_ROUTED0(FrameMsg_BeforeUnload)
 
 // Instructs the frame to swap out for a cross-site transition, including
-// running the unload event handler. Expects a SwapOut_ACK message when
-// finished.
-IPC_MESSAGE_ROUTED0(FrameMsg_SwapOut)
+// running the unload event handler and creating a RenderFrameProxy with the
+// given |proxy_routing_id|. Expects a SwapOut_ACK message when finished.
+IPC_MESSAGE_ROUTED1(FrameMsg_SwapOut,
+                    int /* proxy_routing_id */)
 
 // Request for the renderer to insert CSS into the frame.
 IPC_MESSAGE_ROUTED1(FrameMsg_CSSInsertRequest,
                     std::string  /* css */)
 
 // Request for the renderer to execute JavaScript in the frame's context.
 //
 // javascript is the string containing the JavaScript to be executed in the
 // target frame's context.
 //
@@ skipping 29 matching lines @@
 // doing so.
 IPC_MESSAGE_ROUTED1(FrameMsg_Reload,
                     bool /* ignore_cache */)
 
 // Notifies the color chooser client that the user selected a color.
 IPC_MESSAGE_ROUTED2(FrameMsg_DidChooseColorResponse, unsigned, SkColor)
 
 // Notifies the color chooser client that the color chooser has ended.
 IPC_MESSAGE_ROUTED1(FrameMsg_DidEndColorChooser, unsigned)
 
+// Notifies the corresponding RenderFrameProxy object to replace itself with the
+// RenderFrame object it is associated with.
+IPC_MESSAGE_ROUTED0(FrameMsg_DeleteProxy)

[ncarter (slow), 2014/05/16 20:56:17]

Since this is routed to a RenderFrameProxy, should this message be named
FrameProxyMsg_Delete instead of FrameMsg_DeleteProxy ?

[nasko, 2014/05/16 23:10:42]

Probably. I've avoided introducing Proxy prefix for now, as a bunch of other
IPCs are now routed through the proxy objects as well. I'll do a rename pass
later on.

+
 // -----------------------------------------------------------------------------
 // Messages sent from the renderer to the browser.
 
 // Blink and JavaScript error messages to log to the console
 // or debugger UI.
 IPC_MESSAGE_ROUTED4(FrameHostMsg_AddMessageToConsole,
                     int32, /* log level */
                     base::string16, /* msg */
                     int32, /* line number */
                     base::string16 /* source id */ )
@@ skipping 228 matching lines @@
                     SkColor /* color */)
 
 // Notifies the browser that media has started/stopped playing.
 IPC_MESSAGE_ROUTED3(FrameHostMsg_MediaPlayingNotification,
                     int64 /* player_cookie, distinguishes instances */,
                     bool /* has_video */,
                     bool /* has_audio */)
 
 IPC_MESSAGE_ROUTED1(FrameHostMsg_MediaPausedNotification,
                     int64 /* player_cookie, distinguishes instances */)