Fix leaking of RenderFrames.

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 "cc/surfaces/surface_id.h"
 #include "cc/surfaces/surface_sequence.h"
 #include "content/common/content_export.h"
@@ skipping 492 matching lines @@
 // disowned.
 IPC_MESSAGE_ROUTED1(FrameMsg_UpdateOpener, int /* opener_routing_id */)
 
 // Requests that the RenderFrame send back a response after waiting for the
 // commit, activation and frame swap of the current DOM tree in blink.
 IPC_MESSAGE_ROUTED1(FrameMsg_VisualStateRequest, uint64 /* id */)
 
 // Instructs the renderer to create a new RenderFrame object.
 IPC_MESSAGE_CONTROL1(FrameMsg_NewFrame, FrameMsg_NewFrame_Params /* params */)
 
+// Instructs the renderer to detach the RenderFrame object identified by
+// |routing_id|.
+IPC_MESSAGE_CONTROL1(FrameMsg_Detach, int /* routing_id */)

[dcheng, 2015/12/11 23:40:17]

I think I would personally see this as a routed message: I see it as reuse of
the existing routing machinery.

In spirit, this is pretty similar to RenderWidget::OnClose(), which essentially
self-deletes (by calling Release()).

[nasko, 2015/12/14 21:00:02]

Done.

+
 // Instructs the renderer to create a new RenderFrameProxy object with
 // |routing_id|.  |render_view_routing_id| identifies the
 // RenderView to be associated with this proxy.  The new proxy's opener should
 // be set to the object identified by |opener_routing_id|, or to null if that
 // is MSG_ROUTING_NONE.  The new proxy should be created as a child of the
 // object identified by |parent_routing_id| or as top level if that is
 // MSG_ROUTING_NONE.
 IPC_MESSAGE_CONTROL5(FrameMsg_NewFrameProxy,
                      int /* routing_id */,
                      int /* render_view_routing_id */,
@@ skipping 753 matching lines @@
 IPC_MESSAGE_ROUTED1(FrameHostMsg_ShowPopup,
                     FrameHostMsg_ShowPopup_Params)
 IPC_MESSAGE_ROUTED0(FrameHostMsg_HidePopup)
 
 #endif
 
 // Adding a new message? Stick to the sort order above: first platform
 // independent FrameMsg, then ifdefs for platform specific FrameMsg, then
 // platform independent FrameHostMsg, then ifdefs for platform specific
 // FrameHostMsg.