Commit 562598cd authored by Minoru Chikamune's avatar Minoru Chikamune Committed by Chromium LUCI CQ

Fix "--disable-hang-monitor" command line flag

Before this CL, HungRendererDialogView ("Page Unresponsive") dialog was shown even if "--disable-hang-monitor" command line flag was specified.

After this CL, the command line will work as expected. This switch is useful when you use debugger.

Bug: 1161698
Change-Id: I19089305a62e31d396cd1a69aef86a08cbc301ad
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2602957Reviewed-by: default avatarAvi Drissman <avi@chromium.org>
Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
Reviewed-by: default avatarKouhei Ueno <kouhei@chromium.org>
Commit-Queue: Minoru Chikamune <chikamune@chromium.org>
Cr-Commit-Position: refs/heads/master@{#839406}
parent 301c6e94
...@@ -3957,6 +3957,11 @@ bool WebContentsImpl::IsJavaScriptDialogShowing() const { ...@@ -3957,6 +3957,11 @@ bool WebContentsImpl::IsJavaScriptDialogShowing() const {
} }
bool WebContentsImpl::ShouldIgnoreUnresponsiveRenderer() { bool WebContentsImpl::ShouldIgnoreUnresponsiveRenderer() {
// Suppress unresponsive renderers if the command line asks for it.
if (base::CommandLine::ForCurrentProcess()->HasSwitch(
switches::kDisableHangMonitor))
return true;
if (suppress_unresponsive_renderer_count_ > 0) if (suppress_unresponsive_renderer_count_ > 0)
return true; 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