Fix leaking of RenderFrames.

content/common/frame_messages.h

Index: content/common/frame_messages.h
diff --git a/content/common/frame_messages.h b/content/common/frame_messages.h
index f9604de61e7290ca24e7d5b19a72b7c0f7372c51..ca1d22009483899b09811a5fd0ecc86a08ccb437 100644
--- a/content/common/frame_messages.h
+++ b/content/common/frame_messages.h
@@ -510,6 +510,10 @@ 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