Issue metadata
Sign in to add a comment
|
Newly spawned renderers briefly throttle timers before becoming visible |
||||||||||||||||||||||||
Issue descriptionWhen a new renderer is spawned, its render widget is made hidden until some time after the navigation is committed (?). The means timer tasks can be briefly throttled, which could have harm page load time. Debug log from navigating to a new site from the omnibox: RendererSchedulerImpl 0x141af17d9000 WebFrameScheduler 0x141af1963a80 WebFrameScheduler 0x141af1963a80 page visible: 1 WebFrameScheduler 0x141af1963a80 page visible: 0 WebFrameScheduler 0x141af1963a80 throttling timers RendererSchedulerImpl 0x141af17d9000 is hidden RendererSchedulerImpl 0x141af17d9000 is hidden WebFrameScheduler 0x141af1963a80 page visible: 1 Here page hiding is triggered by ViewMsg_WasHidden.
,
Dec 7 2016
Re #1: Sounds reasonable.
,
Dec 7 2016
We also talked about a more near term workaround where we have a ~10 second grace period before we apply throttling based on a visibility change (which will be useful for other reasons too).
,
Dec 7 2016
The challenge is to solve this in a way that doesn't harm the browser restore case. In that case we have, let's say, twenty tabs all trying to be restored simultaneously and we want the visible one to run at full priority while all of the others run at lower priority with throttled timers. It seems like the correct solution is to count a newly spawned renderer as visible if the tab that it will show up in is visible. I don't know what is involved in making this work.
,
Dec 8 2016
Agreed. I think we need the global tab coordinator in place before we can do prioritization like that.
,
Dec 11 2017
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 11 2017
This is effectively a duplicate of bug 560446 . I'm going to close it as such and annotate that bug to remind those working on it of this aspect of the issue. |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by skyos...@chromium.org
, Dec 7 2016