1618646 - Minor ColumnSet cleanup

Status: RESOLVED FIXED

(Core :: Layout: Columns, task)

Comment 1

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

Attachment: Bug 1618646 Part 1 - Remove nsColumnSetFrame::GetAvailableContentBSize().

ColumnSet is an inner frame now. It cannot have border and padding.

Comment 2

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

Attachment: Bug 1618646 Part 2 - Remove mComputedBSize and mConsumedBSize in nsColumnSetFrame::ReflowConfig.

They were used when nsColumnSetFrame was still a column container, but
are not used today.

Depends on D64646

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3b1c588d756c
Part 1 - Remove nsColumnSetFrame::GetAvailableContentBSize(). r=mats
https://hg.mozilla.org/integration/autoland/rev/e7699efb2aae
Part 2 - Remove mComputedBSize and mConsumedBSize in nsColumnSetFrame::ReflowConfig. r=mats

Comment 4

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/3b1c588d756c
https://hg.mozilla.org/mozilla-central/rev/e7699efb2aae

Comment 5

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

Attachment: Bug 1618646 followup - Remove the declaration of nsColumnSetFrame::GetAvailableContentBSize(). r=mats

This should've been removed in Part 1.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/208adff7b4a3
followup - Remove the declaration of nsColumnSetFrame::GetAvailableContentBSize(). r=mats

Comment 7

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/208adff7b4a3