1866836 - Preparation to fix table available block-size

Status: RESOLVED FIXED

(Core :: Layout: Tables, task)

Description

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

This bug is a preparation / refactor to fix bug 1648947. These patches do not change behavior.

Comment 1

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

Attachment: Bug 1866836 Part 1 - Consolidate available size computation in TableReflowInput's constructor.

Before this patch, we get available block-size in Reflow() and available
inline-size in ReflowTable(). Then we adjust them in TableReflowInput's
constructor. This patch moves all the computation into TableReflowInput's
constructor.

This is a preparation for bug 1863421, and doesn't change behavior.

Comment 2

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

Attachment: Bug 1866836 Part 2 - Pass border and padding down from nsTableFrame::Reflow().

In Bug 1863421, we'll apply ApplySkipSides() for borderPadding. This patch
ensures we use the correct skipped border and padding everywhere.

This patch doesn't change behavior.

Depends on D194788

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/845986cc0a8d
Part 1 - Consolidate available size computation in TableReflowInput's constructor. r=layout-reviewers,emilio
https://hg.mozilla.org/integration/autoland/rev/90d0849cf4a3
Part 2 - Pass border and padding down from nsTableFrame::Reflow(). r=layout-reviewers,emilio

Comment 4

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/845986cc0a8d
https://hg.mozilla.org/mozilla-central/rev/90d0849cf4a3