[chromium] Rework page scale factor limits initialization
Created attachment 185727 [details]
Comment on attachment 185727 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=185727&action=review
> + if (view && view->needsLayout())
> + view->layout();
Why doesn't this cause an infinite loop?
Page scale factor limits are supposed to converge and it shouldn't set needsLayout() repeatedly. What I expect to cause a layout right now is fixed-position element positioning, and that doesn't affect the inputs of computePageScaleFactorLimits. If there's a bug and it doesn't converge, that would cause problems anyway and it's probably better to catch that issue via crash report than via reports of glitchniness or of very slow renderer due to repeated layout.
Committed r141483: <http://trac.webkit.org/changeset/141483>