Please attempt to scale this page to repo. Notice that the top Ad is not always correctly scaled.
Created attachment 112420 [details] Reduction I've attached a reduction for this bug. This happens on Chromium Linux (with fixed layout and scaling enabled locally) and may or may not repo on other platforms (not tested yet): 1. Open up iframe-content-scaling-bug.html. 2. Change the scale factor of the page 3. Reload The two green boxes should be the same size, but the green box in the iframe ends up double scaling.
Created attachment 112422 [details] Fixed Reduction
I'm beginning to suspect this bug and this bug: https://bugs.webkit.org/show_bug.cgi?id=70632 might be related because this reduction is causing Chromium to crash occasionally.
I believe I have a fix for this: Changing: documentStyle->setPageScaleTransform(document->page() ? document->page()->pageScaleFactor() : 1); to documentStyle->setPageScaleTransform(frame ? frame->frameScaleFactor() : 1); in CSSStyleSelector::styleForDocument fixes the issue. Now I just have to come up with an automated layout test for this.
(In reply to comment #4) > I believe I have a fix for this: > > Changing: > > documentStyle->setPageScaleTransform(document->page() ? document->page()->pageScaleFactor() : 1); > > to > > documentStyle->setPageScaleTransform(frame ? frame->frameScaleFactor() : 1); > > in CSSStyleSelector::styleForDocument > > fixes the issue. Now I just have to come up with an automated layout test for this. It seems it's difficult to write an automated layout test to repo this issue. I can only repro it in the browser if I reload the page after I set the page scale factor.
Created attachment 112710 [details] Patch
Comment on attachment 112710 [details] Patch Clearing flags on attachment: 112710 Committed r98637: <http://trac.webkit.org/changeset/98637>
All reviewed patches have been landed. Closing bug.