New issue
Advanced search Search tips

Issue 800115 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

Clean up RPH logic dealing with active views to consider pending views when needed

Project Member Reported by alex...@chromium.org, Jan 8 2018

Issue description

I'm filing this to track the followup work mentioned in https://chromium-review.googlesource.com/c/chromium/src/+/852982#message-b9b9df3b20a5c9369a76561c47425fc6ea5fe5ef, which was fixing  issue 799399 .  In particular, we need to:

- Update comments on FastShutdownIfPossible to also mention pending views.

- Look at whether chrome::CloseWebContentses is correct when it counts the numbers of tabs in the presence of OOPIFs and/or process reuse.

- Fix MockRenderProcessHost to also properly use pending views.

- See if RFHI::AllowBindings also has a bug with pending views.  Currently it checks "GetProcess()->GetActiveViewCount() > 1".
 

Sign in to add a comment