Bug 266838

Summary: Refactor RenderGrid::computeContentPositionAndDistributionOffset and contentDistributionOffset
Product: WebKit Reporter: fantasai <fantasai.bugs>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: Safari 17   
Hardware: Unspecified   
OS: Unspecified   

Description fantasai 2023-12-22 23:32:21 PST 
These functions are doing weird things like:
* Treating negative offsets as flagging an invalid state even though offsets can validly be negative.
* Letting control flow through in some cases without an obvious initialization to zero, but then insisting on setting things to zero in other cases... like either we're starting at zero and don't need to set it, or we need to set things to zero, pick one.
Comment 1 Radar WebKit Bug Importer 2023-12-22 23:32:33 PST 
<rdar://problem/120064485>
Comment 2 EWS 2024-01-02 19:23:27 PST 
Committed 272591@main (9c0b2b0c72a4): <https://commits.webkit.org/272591@main>

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