Commit dde83b31 authored by danakj's avatar danakj Committed by Commit Bot

Remove WidgetHostMsgs from SwappedOutMessages.

RenderViewHostImpl checks with SwappedOutMessages to receive IPCs. The
list of allowed IPCs include some WidgetHostMsgs. However
WidgetHostMsgs have a separate routing id that go to the
RenderWidgetHostImpl, which does not use SwappedOutMessages and does
not drop any IPCs. So the WidgetHostMsgs do not need to be listed in
the SwappedOutMessages allow list, they do nothing there.

R=creis@chromium.org
TBR=presubmit is broken

Bug: 419087, 745091
Change-Id: I7f19af25e97973da7b3d2c6508acb3058f21539e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1693802
Commit-Queue: danakj <danakj@chromium.org>
Reviewed-by: default avatarAlex Moshchuk <alexmos@chromium.org>
Reviewed-by: default avatarCharlie Reis <creis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#676100}
parent 6faba39b
......@@ -5,9 +5,7 @@
#include "content/common/swapped_out_messages.h"
#include "content/common/accessibility_messages.h"
#include "content/common/frame_messages.h"
#include "content/common/view_messages.h"
#include "content/common/widget_messages.h"
#include "content/public/common/content_client.h"
namespace content {
......@@ -49,10 +47,6 @@ bool SwappedOutMessages::CanHandleWhileSwappedOut(
// Note that synchronous messages that are not handled will receive an
// error reply instead, to avoid leaving the renderer in a stuck state.
switch (msg.type()) {
// We allow closing even if we are in the process of swapping out.
case WidgetHostMsg_Close::ID:
// Sends an ACK.
case WidgetHostMsg_RequestSetBounds::ID:
// Sends an ACK.
case AccessibilityHostMsg_EventBundle::ID:
return true;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment