Issue metadata
Sign in to add a comment
|
Resizing usatoday.com recomputes the entire page style and layout because of media queries |
||||||||||||||||||||||||
Issue descriptionChrome Version : 57.0.2986.0 What steps will reproduce the problem? 1. Load http://www.usatoday.com/ 2. Resize the window Attached is a trace taken by vmiura@. It shows an 8ms updateActiveStyles() call (most time is spend in RuleSet::addRulesFromSheet) followed by a ~2700 element recalc style which is the result of toggling on and off matching media queries. Afterward we have a giant ~5600 element layout. Can we make this recalcStyle and updateActiveStyles faster? Are we doing over invalidation of style here? If we can make the recalcStyle fast then we can pass off the slow layout costs to Layout team and see if LayoutNG or targeted fixes can improve the speed there. (Note this is a continuation of issue 570166 where raster was addressed by turning on Ganesh.)
,
Jan 21 2017
I'm also seeing really slow layout on cnn.com desktop site running on nexus 5. Not sure if it's the same thing. Loading the page for the first time is really slow (10s of seconds) and so is tap to zoom. However, pinch to zoom seems to work well, probably because the main thread is not involved in that case. Trace attached.
,
Jan 24 2017
,
Jan 24 2017
Ignore my previous comment. I was running a debug build. <shame cube>
,
Feb 12 2017
,
Apr 10 2017
Is this related to bug 570165 ?
,
Apr 10 2017
Yeah I think these are probably the same bug.
,
Apr 10 2017
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by esprehn@chromium.org
, Jan 20 2017