1569701 - Use GetEffectiveComputedBSize() when applying ColumnSetWrapper's block-size constraint for ColumnSet

Status: RESOLVED FIXED

(Core :: Layout: Columns, task, P3)

Description

[Ting-Yu Lin [:TYLin] (PST, UTC-7)]

Bug 1523595 Part 3 applied ColumnSetWrapper's block-size constraint to the available size when creating BlockReflowInput for ColumnSet, but it didn't consider the case when theColumnSetWrapper is fragmented. We might want to use GetEffectiveComputedBSize to get the remaining block-size for ColumnSetWrapper.

Comment 1

[Ting-Yu Lin [:TYLin] (PST, UTC-7)]

Attachment: Bug 1569701 Part 0 - Use nscoord instead of int for contentBEnd in nsColumnSetFrame::ReflowChildren().

Comment 2

[Ting-Yu Lin [:TYLin] (PST, UTC-7)]

Attachment: Bug 1569701 Part 1 - Make BlockReflowInput::mConsumedBSize a constant, and delete it assessor method.

We always pass consumed block-size into BlockReflowInput's constructor
in nsBlockFrame::Reflow(). By making mConsumedBSize a constant, its
assessor method becomes redundant.

Update the documentation to reflect the reality that ConsumedBSize()
accumulates content block-size from all previous continuations, which
was done in Bug 1506293 Part 2.

Depends on D41905

Comment 3

[Ting-Yu Lin [:TYLin] (PST, UTC-7)]

Attachment: Bug 1569701 Part 2 - Use ColumnSetWrapper's effective computed content block-size to impose constraint for ColumnSet.

Run broken-column-rule-1.html with column-span enabled because it was
regressed by Bug 1548100 Part 2, but fixed by this patch.

Depends on D41906

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e56e6ed91e5a
Part 0 - Use nscoord instead of int for contentBEnd in nsColumnSetFrame::ReflowChildren(). r=dbaron
https://hg.mozilla.org/integration/autoland/rev/6d5655c8638d
Part 1 - Make BlockReflowInput::mConsumedBSize a constant, and delete it assessor method. r=dbaron
https://hg.mozilla.org/integration/autoland/rev/e9611b9cff9a
Part 2 - Use ColumnSetWrapper's effective computed content block-size to impose constraint for ColumnSet. r=dbaron

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/18462 for changes under testing/web-platform/tests

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 7

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/e56e6ed91e5a
https://hg.mozilla.org/mozilla-central/rev/6d5655c8638d
https://hg.mozilla.org/mozilla-central/rev/e9611b9cff9a