Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 90bcb7bdb7bd5f7c45f069e0e9b612a0dac58f39
      
https://github.com/WebKit/WebKit/commit/90bcb7bdb7bd5f7c45f069e0e9b612a0dac58f39
  Author: Sammy Gill <[email protected]>
  Date:   2026-02-13 (Fri, 13 Feb 2026)

  Changed paths:
    M Source/WebCore/layout/formattingContexts/grid/GridFormattingContext.cpp
    M Source/WebCore/layout/formattingContexts/grid/GridFormattingContext.h
    M Source/WebCore/layout/formattingContexts/grid/GridLayoutUtils.cpp
    M Source/WebCore/layout/formattingContexts/grid/GridLayoutUtils.h
    M Source/WebCore/layout/formattingContexts/grid/TrackSizingAlgorithm.cpp
    M Source/WebCore/layout/integration/grid/LayoutIntegrationGridLayout.cpp
    M Source/WebCore/layout/integration/grid/LayoutIntegrationGridLayout.h

  Log Message:
  -----------
  [GFC][Integration] min and max content height of the grid is size of rows and 
gutters
https://bugs.webkit.org/show_bug.cgi?id=307713
rdar://170267496

Reviewed by Alan Baradlay.

The spec defines the min/max content height of the grid as the sum of
the rows and gutters.
https://drafts.csswg.org/css-grid-1/#intrinsic-sizes

The track sizes are outputted as part of GridLayout, so we need to make
sure that this is reflected on RenderGrid. This is because RenderGrid
treats height: auto always as max-content barring any overriding sizes
or something else computed by 
RenderGrid::availableLogicalHeightForContentBox/RenderGrid::hasDefiniteLogicalHeight.

So if we see a MinContent or MaxContent constraint in the block axis,
which was determined by constraintsForGridContent, we should sum the
rows and gutters to make sure RenderGrid has its height set properly.

* Source/WebCore/layout/formattingContexts/grid/GridLayoutUtils.cpp:
(WebCore::Layout::GridLayoutUtils::totalGuttersSize):
Make this available to use outside of TrackSizingAlgorithm.

Canonical link: https://commits.webkit.org/307492@main



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications

Reply via email to