Issue metadata
Sign in to add a comment
|
Tab loading spinner animation appears to be janking due to compositor frame timeout |
||||||||||||||||||||||
Issue descriptionChrome Version: 65.0.3325.9 OS: ChromeOS What steps will reproduce the problem? (1) Open a web site that contains links that will open in a new tab, e.g. GMail. (2) Click on one of the links. (3) Watch the tab spinner while the new tab is opening. What is the expected result? Expect that the spinner animates smoothly. What happens instead? Spinner often (seems dependent on the site, or possibly just on how long it takes to load) animates at a lower-than-normal frame rate - this looks like issues we've had previously with the compositor "timing out" waiting for a renderer to provide a suitable frame. I hit a similar issue recently, also on ChromeOS, where the desktop wallpaper on one monitor had stayed blurred after unlocking the workstation, and doing anything which caused the single window on that monitor to be re-composited caused things to be janky. e.g. if I dragged a second window, wholly obscuring the original one, then things were smooth, but if any of the original window was visible then the frame rate would drop to this janky-looking level. I'm also able to repro the spinner jank by just opening chromium-reviews in a new tab, FWIW.
,
Jan 26 2018
Saman is this related to your recent change to navigation?
,
Jan 26 2018
Opening a tab that doesn't have contents always triggered synchronization, but I guess now we do an extra timeout after navigation so things might have gotten worse?
,
Jan 26 2018
We should start using customizable deadlines as soon as possible.
,
Jan 26 2018
,
Jan 26 2018
I'll wire up custom deadlines today. I hope the bots are fixed soon :(
,
Jan 27 2018
Just repro'd this again on ChromeOS 65.0.3322.0 (Official Build) dev (32-bit) - seems to repro reasonably reliably while loading chrome://system
,
Jan 27 2018
I'm seeing this as well and it should be fixable now. I'll have a CL ready shortly.
,
Jan 27 2018
I have a WIP patch that addresses this issue by reducing the synchronization deadline on navigation: https://chromium-review.googlesource.com/c/chromium/src/+/890242
,
Jan 30 2018
The CL has landed. Tab spinner jank should be gone. https://chromium-review.googlesource.com/c/chromium/src/+/890242
,
Jan 30 2018
,
Feb 6 2018
I'm marking as FIXED. wez@: Please reopen if you still feel there's a problem here. Thanks! |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by samans@chromium.org
, Jan 26 2018