Remove forwarding of RenderView messages to RenderWidget.

This was already effectively disabled, but retained as a CHECK() to
ensure it wasn't happening in the wild. It doesn't appear to be, so
remove it altogether.

Bug: 990607
Change-Id: I8635aa233771395eb57e7ecc67b0cd4bd5686feb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1747155
Reviewed-by: Erik Chen <erikchen@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#685846}

content/renderer/render_view_impl.cc

@@ -1302,10 +1302,6 @@ bool RenderViewImpl::OnMessageReceived(const IPC::Message& message) {
 
     // Adding a new message? Add platform independent ones first, then put the
     // platform specific ones at the end.
-
-    // Have the widget handle all other messages.
-    // TODO(ajwong): Remove this cross-object dispatch.
-    IPC_MESSAGE_UNHANDLED(handled = ForwardToWidgetAndCheckIfHandled(message))
   IPC_END_MESSAGE_MAP()
 
   return handled;
@@ -2273,13 +2269,4 @@ RenderViewImpl::GetCleanupTaskRunner() {
       ->CleanupTaskRunner();
 }
 
-bool RenderViewImpl::ForwardToWidgetAndCheckIfHandled(
-    const IPC::Message& message) {
-  // TODO(https://crbug.com/990607): This CHECK is temporary and will be
-  // removed, assuming nothing crashes in the wild.
-  bool handled = GetWidget()->OnMessageReceived(message);
-  CHECK(!handled);
-  return handled;
-}
-
 }  // namespace content

content/renderer/render_view_impl.h

@@ -516,8 +516,6 @@ class CONTENT_EXPORT RenderViewImpl : public blink::WebViewClient,
   void UpdateThemePrefs() {}
 #endif
 
-  bool ForwardToWidgetAndCheckIfHandled(const IPC::Message& message);
-
   // ---------------------------------------------------------------------------
   // ADDING NEW FUNCTIONS? Please keep private functions alphabetized and put
   // it in the same order in the .cc file as it was in the header.