1979806 - More minor cleanups in grid sizing code

Status: RESOLVED FIXED

(Core :: Layout: Grid, task)

Description

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

A sequel to Bug 1979643. I also noticed minor cleanup opportunities while working on the grid sizing code.

Comment 1

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

Attachment: Bug 1979806 - Allow clang-format for TrackSize::StateBits. r?#layout

I feel it is better for TrackSize::StateBits be formatted automatically by
clang-format.

This patch removes the last clang-format off directive in the grid container.
Previously, we removed one for GridItemInfo::StateBits in Bug 1878846:
https://hg.mozilla.org/mozilla-central/rev/7829576fd77e

Comment 2

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

Attachment: Bug 1979806 - Dump "auto" for eAutoMinSizing bit in TrackSize::StateBits. r?#layout

This patch dumps "auto" instead "auto-min" for eAutoMinSizing bit, maintaining
symmetry with how the eAutoMaxSizing bit is dumped.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/1f5d8f573c58
https://hg.mozilla.org/integration/autoland/rev/533ed5209255
Allow clang-format for TrackSize::StateBits. r=layout-reviewers,emilio
https://github.com/mozilla-firefox/firefox/commit/e8c9667a628c
https://hg.mozilla.org/integration/autoland/rev/7cb9dfae673e
Dump "auto" for eAutoMinSizing bit in TrackSize::StateBits. r=layout-reviewers,emilio

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/533ed5209255
https://hg.mozilla.org/mozilla-central/rev/7cb9dfae673e