Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: a5fd3ffea09db8f5b5c3365658b1ee28b5e1e718
https://github.com/WebKit/WebKit/commit/a5fd3ffea09db8f5b5c3365658b1ee28b5e1e718
Author: Alan Baradlay <[email protected]>
Date: 2026-07-19 (Sun, 19 Jul 2026)
Changed paths:
M Source/WebCore/rendering/RenderFlexLayout.cpp
M Source/WebCore/rendering/RenderFlexLayout.h
Log Message:
-----------
[cleanup] Name FlexLayout's per-item and per-line list types after the LFC
FlexLayout aliases
https://bugs.webkit.org/show_bug.cgi?id=318707
Reviewed by Antti Koivisto.
FlexLayout spelled its per-item and per-line lists as raw
Vector<LayoutUnit>/Vector<LayoutPoint> everywhere,
while the LFC Layout::FlexLayout the legacy path is converging on names them
SizeList / PositionList /
LinesCrossSizeList / LinesCrossPositionList. Introduce the same aliases (next
to the existing LineRanges and
FlexBaseAndHypotheticalMainSizeList) and use them across the signatures and
layout() locals.
The aliases resolve to Vector, not FFC's FixedVector, because the legacy
pipeline resizes and rewrites these
mid-algorithm (distribute rewrites the main-size list, the cross-axis phases
push into the position lists). A
few raw uses are left deliberately: FlexLines::hypotheticalMainSizes (a
per-line main size, which FFC has no
alias for), the flexItemsCrossOffsetList accumulator in
performBaselineAlignment, and the std::span main-size
view in resolveFlexibleLengthsForLineItems.
No change in behavior.
* Source/WebCore/rendering/RenderFlexLayout.cpp:
* Source/WebCore/rendering/RenderFlexLayout.h:
Canonical link: https://commits.webkit.org/317510@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
