New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 598125 link

Starred by 4 users

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Feature

Blocking:
issue 597012



Sign in to add a comment

Throttle document.title updates

Project Member Reported by rsch...@chromium.org, Mar 26 2016

Issue description

Doing it too quickly can hang both the renderer and browser processes. See  issue 597012  for more details.
 
Blocking: 597012
Cc: haraken@chromium.org esprehn@chromium.org
Components: Blink>Architecture
I couldn't find a team that fits this work, so I am throwing this to the architecture pile :)
Cc: jam@chromium.org jochen@chromium.org
In general how do we throttle IPC? I can do this in the renderer, but I wonder if we try to avoid having a runaway renderer be able to DOS the browser process in a more general way.

jam@, jochen@ ?

Comment 5 by jam@chromium.org, Mar 28 2016

We don't have a generic way of doing this; although it's been discussed on and off for mojo ipc. The status quo is that we're waiting to see how a few consumers do this manually, and then that'll help guide what the API should look like. see  bug 584775 , which I just annotated to point back here as an another example.

regarding protecting from an exploited renderer: it's assumed that there's nothing we can do to prevent a renderer from causing OOM in the browser. There are too many heavy operations it can trigger.
While this may also have been leading to OOM, that's a separate issue. I think the concern here is that the IPC flood is just overwhelming the browser process in terms of CPU.

Comment 7 by jam@chromium.org, Mar 28 2016

The same reasoning apply to any resource consumption (memory, cpu, gpu memory, network sockets, file handles etc...) that's requested from the renderer.
Status: Untriaged (was: Unconfirmed)
Marking the above issue as Untriaged as this is a feature request.

Thank you!

Comment 9 by tkent@chromium.org, Jun 23 2016

Components: -Blink>Architecture Blink>Internals
Renaming Blink>Architecture to Blink>Internals

Cc: u...@chromium.org abodenha@chromium.org rsch...@chromium.org
 Issue 597012  has been merged into this issue.
Cc: -rsch...@chromium.org

Sign in to add a comment