1501109 - Simplify flexbox devtools growthState API to just report "growing" or "shrinking" for each line (never "unchanged")

Status: RESOLVED FIXED

(Core :: Layout: Block and Inline, enhancement, P3)

Description

[Daniel Holbert [:dholbert] (PTO until 2/24)]

Spinning this off for the c++/webidl changes for bug 1501066.  See bug 1501066 comment 8.

Basically, we should:
 - assign growthState to "growing" or "shrinking" using the same check that we use to assign isUsingFlexGrow, here:
https://searchfox.org/mozilla-central/rev/fcfb479e6ff63aea017d063faa17877ff750b4e5/layout/generic/nsFlexContainerFrame.cpp#2642-2644
 - remove everything that messes with growthState below that.
 - remove all mentions of "unchanged"

And while we're at it, we might as well get rid of the ComputedFlexLineInfo::GrowthState enum type entirely, and just use mozilla::dom::FlexLineGrowthState.  i.e. we should just jump straight to the final type rather than doing this type-conversion:
https://searchfox.org/mozilla-central/rev/fcfb479e6ff63aea017d063faa17877ff750b4e5/dom/flex/FlexLineValues.cpp#33-43

Brad, I'll presumptuously set assignee=you to get this into your queue (to look at after bug 1498273 probably), though I'm also happy to take it as well if you'd prefer.

Comment 1

[Brad Werth [:bradwerth] | Out until Feb 2025]

Attachment: Bug 1501109 Part 1: Change Flex API growth state to match layout algorithm, and use enums from webidl. r?dholbert!

Comment 2

[Brad Werth [:bradwerth] | Out until Feb 2025]

Attachment: Bug 1501109 Part 2: Update test expectations for flex line growth state. r?dholbert!

Depends on D9454

Comment 3

[Brad Werth [:bradwerth] | Out until Feb 2025]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=384a2c49d164a714113c2de17ce971b438c1b527

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/19472cba78cc
Part 1: Change Flex API growth state to match layout algorithm, and use enums from webidl. r=dholbert
https://hg.mozilla.org/integration/autoland/rev/ec3c914d6136
Part 2: Update test expectations for flex line growth state. r=dholbert

Comment 5

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/19472cba78cc
https://hg.mozilla.org/mozilla-central/rev/ec3c914d6136