Closed Bug 1803863 Opened 1 year ago Closed 1 year ago

Minor clean up for table frame

Categories

(Core :: Layout: Tables, task)

task

Tracking

()

RESOLVED FIXED
109 Branch
Tracking Status
firefox109 --- fixed

People

(Reporter: TYLin, Assigned: TYLin)

Details

Attachments

(3 files)

See the upcoming patches for details.

The helper cannot fail, so we can return the height directly instead of using an
output argument.

Assignee: nobody → aethanyc
Status: NEW → ASSIGNED
  • Remove unnecessary calls to ClearSize() after declaring a fresh ReflowOutput.
  • Use ReflowOutput::SetSize() in nsTableFrame::FixupPositionedTableParts().

Depends on D163762

Pushed by aethanyc@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/94713a5fbadf
Part 1 - Simplify nsTableFrame::SetupHeaderFooterChild(). r=emilio
https://hg.mozilla.org/integration/autoland/rev/f1123b6e0eed
Part 2 - Make TableReflowInput's member vars conform with coding style. r=emilio
https://hg.mozilla.org/integration/autoland/rev/7e2df6f0d1fd
Part 3 - Simplify the usages for ReflowOutput. r=emilio
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch
Regressions: 1805601
No longer regressions: 1805601
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: