[webkit-changes] [WebKit/WebKit] b79f74: [css-masonry] Fix masonry-intrinsic-sizing-rows-00...

[Brandon Stewart]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b79f7439f9580da79e784a83279751fecbb1e401
      https://github.com/WebKit/WebKit/commit/b79f7439f9580da79e784a83279751fecbb1e401
  Author: Brandon Stewart <brandonstewart at apple.com>
  Date:   2024-10-15 (Tue, 15 Oct 2024)

  Changed paths:
    M LayoutTests/TestExpectations
    M LayoutTests/imported/w3c/web-platform-tests/css/css-grid/masonry/tentative/intrinsic-sizing/masonry-intrinsic-sizing-rows-005-expected.html
    M LayoutTests/imported/w3c/web-platform-tests/css/css-grid/masonry/tentative/intrinsic-sizing/masonry-intrinsic-sizing-rows-005-ref.html
    M LayoutTests/imported/w3c/web-platform-tests/css/css-grid/masonry/tentative/intrinsic-sizing/masonry-intrinsic-sizing-rows-005.html

  Log Message:
  -----------
  [css-masonry] Fix masonry-intrinsic-sizing-rows-005.html
https://bugs.webkit.org/show_bug.cgi?id=281438
rdar://problem/137890822

Reviewed by Sammy Gill.

Update test case to use Ahem font as the default font ends up resolving to 27.02 with 3 characters, which messes with the test case track sizes.

During sizing of the grid the min-content calculations will result in extra padding to the grid in the width direction. This needs to be accounted for too.

* LayoutTests/TestExpectations:
* LayoutTests/imported/w3c/web-platform-tests/css/css-grid/masonry/tentative/intrinsic-sizing/masonry-intrinsic-sizing-rows-005-expected.html:
* LayoutTests/imported/w3c/web-platform-tests/css/css-grid/masonry/tentative/intrinsic-sizing/masonry-intrinsic-sizing-rows-005-ref.html:
* LayoutTests/imported/w3c/web-platform-tests/css/css-grid/masonry/tentative/intrinsic-sizing/masonry-intrinsic-sizing-rows-005.html:

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



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