987605 - call MaybeReflowForInflationScreenWidthChange on SetScrollPositionClampingScrollPortSize instead of SetDisplayPortForElement

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Timothy Nikkel (:tnikkel)]

Attachment: patch

Although the displayport will be updated when the screen size changes, the scroll position clamping scroll port size is more closely related to the screen size. It is set to the area visible on screen for content (the name is outdated now and the actual rect is used for more things). It seems like the more natural place to put it to me.

This simplifies the work in bug 957668 a little.

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 8396302 [details] [diff] [review]
patch

>+  // layout depends on the size of the screen.  Since when the size
>+  // of the screen changes, the scroll position clamping scroll port
>+  // size also changes, we hook in the needed updates here rather

As long as you're confident this is true for both Android and B2G, r=dbaron.

The thing to test is that font inflation changes take place when switching the browser between landscape and portrait mode.  (Fonts should be less inflated when the screen is wider, so having a wider screen should fit more text on each line.)

Comment 2

[Timothy Nikkel (:tnikkel)]

We depend on the scroll position clamping scroll port size for other things, for example fixed pos items are reflowed into that size instead of the viewport size, and the innerWidth/Height of the page reflect it too. So if it didn't get updated we would have other bugs, which we should fix.

Comment 3

[Timothy Nikkel (:tnikkel)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d0fed85e8eca

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/d0fed85e8eca

Comment 5

[Timothy Nikkel (:tnikkel)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/0b82b4683b3c