Bug 303746 – Improve readability of StyleLogicalHeight by adopting a switch on statement.

• Status: RESOLVED
• Resolution: FIXED
• Component: Tables

[Karl Dubost]

See Sam Weining comment on https://github.com/WebKit/WebKit/pull/54635#discussion_r2583134648

Please consider using the switchOn() form here. While a bit more verbose, I think it ends up making things more clear.

WTF::switchOn(styleLogicalHeight,
    [&](const Style::PreferredSize::Fixed& fixed) {
        ...
    },
    [&](Style::IsPercentageOrCalc auto const& percentageOrCalc) {
        ...
    },
    [&](const CSS::Keyword::FitContent&) {
        ...
    },
    [&](const CSS::Keyword::WebkitFillAvailable&) {
        ...
    },
    [&](const CSS::Keyword::MinContent&) {
        ...
    },
    [&](const CSS::Keyword::MaxContent&) {
        ...
    },
    [&](const CSS::Keyword::Intrinsic&) {
        ...
    },
    [&](const CSS::Keyword::MinIntrinsic&) {
        ...
    },
    [&](const CSS::Keyword::Auto&) {
        ...
    }
);

[Radar WebKit Bug Importer]

<rdar://problem/166529986>

[zak ridouh]

Pull request: https://github.com/WebKit/WebKit/pull/59062

[EWS]

Committed 317491@main (27059d44ca36): <https://commits.webkit.org/317491@main>

Reviewed commits have been landed. Closing PR #59062 and removing active labels.