Change FrameMsg_ChildFrameProcessGone to mojom

This CL converts FrameMsg_ChildFrameProcessGone to RenderFrameProxy
and removes FrameMsg_ChildFrameProcessGone message.

Bug: 1051137
Change-Id: I55e24ee59d2c15273988f7d1636edfe2f0198f4a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2062010Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Commit-Queue: Julie Kim <jkim@igalia.com>
Cr-Commit-Position: refs/heads/master@{#742536}

content/browser/frame_host/cross_process_frame_connector.cc

@@ -146,8 +146,7 @@ void CrossProcessFrameConnector::RenderProcessGone() {
   if (IsVisible())
     MaybeLogCrash(CrashVisibility::kCrashedWhileVisible);
 
-  frame_proxy_in_parent_renderer_->Send(new FrameMsg_ChildFrameProcessGone(
-      frame_proxy_in_parent_renderer_->GetRoutingID()));
+  frame_proxy_in_parent_renderer_->ChildProcessGone();
 
   auto* parent_view = GetParentRenderWidgetHostView();
   if (parent_view && parent_view->host()->delegate())

content/browser/frame_host/render_frame_proxy_host.cc

@@ -462,6 +462,10 @@ void RenderFrameProxyHost::DidUpdateVisualProperties(
   GetAssociatedRenderFrameProxy()->DidUpdateVisualProperties(metadata);
 }
 
+void RenderFrameProxyHost::ChildProcessGone() {
+  GetAssociatedRenderFrameProxy()->ChildProcessGone();
+}
+
 void RenderFrameProxyHost::OnRouteMessageEvent(
     const FrameMsg_PostMessage_Params& params) {
   RenderFrameHostImpl* target_rfh = frame_tree_node()->current_frame_host();

content/browser/frame_host/render_frame_proxy_host.h

@@ -168,8 +168,8 @@ class CONTENT_EXPORT RenderFrameProxyHost
   // Requests a viz::LocalSurfaceId to disable auto-resize mode from the parent
   // renderer.
   void DisableAutoResize();
-
   void DidUpdateVisualProperties(const cc::RenderFrameMetadata& metadata);
+  void ChildProcessGone();
 
   blink::AssociatedInterfaceProvider* GetRemoteAssociatedInterfacesTesting();
   bool IsInertForTesting();

content/common/frame_messages.h

@@ -523,10 +523,6 @@ IPC_STRUCT_TRAITS_END()
 // -----------------------------------------------------------------------------
 // Messages sent from the browser to the renderer.
 
-// Notifies the embedding frame that the process rendering the child frame's
-// contents has terminated.
-IPC_MESSAGE_ROUTED0(FrameMsg_ChildFrameProcessGone)
-
 // Sent in response to a FrameHostMsg_ContextMenu to let the renderer know that
 // the menu has been closed.
 IPC_MESSAGE_ROUTED1(FrameMsg_ContextMenuClosed,

content/common/frame_proxy.mojom

@@ -19,6 +19,10 @@ interface RenderFrameProxyHost {
 // RenderFrameProxy. Implemented by renderer processes, called by the browser
 // process.
 interface RenderFrameProxy {
+  // Notifies the remote frame that the process rendering the child frame's
+  // contents has terminated.
+  ChildProcessGone();
+
   // This function enables auto-resize mode from the parent renderer.
   EnableAutoResize(gfx.mojom.Size min_size, gfx.mojom.Size max_size);
 

content/renderer/render_frame_proxy.cc

@@ -374,7 +374,6 @@ bool RenderFrameProxy::OnMessageReceived(const IPC::Message& msg) {
 
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(RenderFrameProxy, msg)
-    IPC_MESSAGE_HANDLER(FrameMsg_ChildFrameProcessGone, OnChildFrameProcessGone)
     IPC_MESSAGE_HANDLER(FrameMsg_UpdateOpener, OnUpdateOpener)
     IPC_MESSAGE_HANDLER(FrameMsg_ViewChanged, OnViewChanged)
     IPC_MESSAGE_HANDLER(FrameMsg_DidUpdateName, OnDidUpdateName)
@@ -413,7 +412,7 @@ void RenderFrameProxy::OnDeleteProxy() {
   web_frame_->Detach();
 }
 
-void RenderFrameProxy::OnChildFrameProcessGone() {
+void RenderFrameProxy::ChildProcessGone() {
   crashed_ = true;
   compositing_helper_->ChildFrameGone(local_frame_size(),
                                       screen_info().device_scale_factor);

content/renderer/render_frame_proxy.h

@@ -221,7 +221,6 @@ class CONTENT_EXPORT RenderFrameProxy : public IPC::Listener,
 
   // IPC handlers
   void OnDeleteProxy();
-  void OnChildFrameProcessGone();
   void OnCompositorFrameSwapped(const IPC::Message& message);
   void OnUpdateOpener(int opener_routing_id);
   void OnViewChanged(const FrameMsg_ViewChanged_Params& params);
@@ -236,6 +235,7 @@ class CONTENT_EXPORT RenderFrameProxy : public IPC::Listener,
   void DisableAutoResize() override;
   void DidUpdateVisualProperties(
       const cc::RenderFrameMetadata& metadata) override;
+  void ChildProcessGone() override;
 
   // ChildFrameCompositor:
   cc::Layer* GetLayer() override;