Commit 2dc56dc4 authored by David Bokan's avatar David Bokan Committed by Commit Bot

Fix crash in event sender

This is a small amendment to https://crrev.com/c/1763375. The null check
added there should go before the DCHECK.

Bug: 997430
Change-Id: Iabdef43de616b6207516b7efff1c6ecd8e81de81
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1771621
Auto-Submit: David Bokan <bokan@chromium.org>
Reviewed-by: default avatarNasko Oskov <nasko@chromium.org>
Commit-Queue: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690986}
parent ccd6a0c1
...@@ -2844,11 +2844,11 @@ blink::WebWidget* EventSender::widget() { ...@@ -2844,11 +2844,11 @@ blink::WebWidget* EventSender::widget() {
} }
blink::WebFrameWidget* EventSender::mainFrameWidget() { blink::WebFrameWidget* EventSender::mainFrameWidget() {
if (!view() || !view()->MainFrame())
return nullptr;
DCHECK(view()->MainFrame()->IsWebLocalFrame()) DCHECK(view()->MainFrame()->IsWebLocalFrame())
<< "Event Sender doesn't support being run in a remote frame for this " << "Event Sender doesn't support being run in a remote frame for this "
"operation."; "operation.";
if (!view() || !view()->MainFrame())
return nullptr;
return view()->MainFrame()->ToWebLocalFrame()->FrameWidget(); return view()->MainFrame()->ToWebLocalFrame()->FrameWidget();
} }
......
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