Bug 1561227 Comment 11 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

I've found another way to solve this bug; possibly a better way. Full-page zoom is triggering an unnecessary resolution change in ShrinkDisplaySizeIfNeeded->UpdateResolution due to aViewportInfo.IsDefaultZoomValid() being true at https://searchfox.org/mozilla-central/source/layout/base/MobileViewportManager.cpp#389. IsDefaultZoomValid() is true because initial-scale is specified. But that shouldn't make content-fitting behave differently. So checking IsDefaultZoomValid() here is probably incorrect and we need something more specific for the intended cases.
I've found another way to solve this bug; possibly a better way. Full-page zoom is triggering an unnecessary resolution change in ShrinkDisplaySizeIfNeeded->UpdateResolution due to aViewportInfo.IsDefaultZoomValid() being true at https://searchfox.org/mozilla-central/source/layout/base/MobileViewportManager.cpp#389. IsDefaultZoomValid() is true because initial-scale is specified. But that shouldn't make content-fitting behave differently after the initial display of the content. So checking IsDefaultZoomValid() here is probably incorrect and we need something more specific for the intended cases.

Back to Bug 1561227 Comment 11