New issue
Advanced search Search tips

Issue 888120 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Cleanup WebContentsImpl::ShouldIgnoreUnresponsiveRenderer

Project Member Reported by lukasza@chromium.org, Sep 21

Issue description

This bug is a follow-up to the CR feedback at https://chromium-review.googlesource.com/c/chromium/src/+/1222684/2/content/browser/web_contents/web_contents_impl.cc#5950

We should split WebContentsImpl::ShouldIgnoreUnresponsiveRenderer into

1) ShouldIgnoreTimeoutsInRenderer
   (to be used via RVHDelegate from
    RenderViewHostImpl::ClosePageTimeout and
    RenderFrameHostImpl::BeforeUnloadTimeout)

2) ShouldIgnoreUnresponsiveRenderer
   (to be used from WebContentsImpl::RendererUnresponsive)
 

Sign in to add a comment