Commit 2015ee53 authored by Dave Tapuska's avatar Dave Tapuska Committed by Commit Bot

Reset the (Local|Remote)Frame receiver on detach.

We don't want to receive messages after we are detached so ensure
we reset the receiver on detach.

BUG=1020551

Change-Id: Iddc8a8a9e1e8f9e998845c876e544c0b770cdfe6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1894918
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: default avatarDaniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#711834}
parent 8a030dd3
......@@ -368,6 +368,7 @@ void LocalFrame::DetachImpl(FrameDetachType type) {
supplements_.clear();
frame_scheduler_.reset();
receiver_.reset();
WeakIdentifierMap<LocalFrame>::NotifyObjectDestroyed(this);
}
......
......@@ -160,6 +160,7 @@ void RemoteFrame::DetachImpl(FrameDetachType type) {
To<RemoteDOMWindow>(dom_window_.Get())->FrameDetached();
if (cc_layer_)
SetCcLayer(nullptr, false, false);
receiver_.reset();
}
bool RemoteFrame::DetachDocument() {
......
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