Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: efacbbdcc42539c57f1fe9c5dcf40251ce703eef
      
https://github.com/WebKit/WebKit/commit/efacbbdcc42539c57f1fe9c5dcf40251ce703eef
  Author: Elika Etemad <[email protected]>
  Date:   2024-07-10 (Wed, 10 Jul 2024)

  Changed paths:
    A LayoutTests/fast/overflow/flexbox-abspos-overflow-expected.html
    A LayoutTests/fast/overflow/flexbox-abspos-overflow.html
    A LayoutTests/fast/overflow/flexbox-multiline-overflow-expected.html
    A LayoutTests/fast/overflow/flexbox-multiline-overflow.html
    M 
LayoutTests/imported/w3c/web-platform-tests/css/css-flexbox/negative-overflow-002-expected.txt
    M 
LayoutTests/platform/ios/imported/w3c/web-platform-tests/css/css-flexbox/negative-overflow-002-expected.txt
    M Source/WebCore/rendering/RenderFlexibleBox.cpp

  Log Message:
  -----------
  Fix error in flexbox alignment overflow calculations.
https://bugs.webkit.org/show_bug.cgi?id=276382
rdar://131201271

Reviewed by Alan Baradlay.

This patch fixes the logic in contentAlignmentStartOverflow correctly
clamp overflow, by handling reversed flows properly rather than returning
an overly-large value in some cases. It also fixes the error where we only
account for the last flex line.

* LayoutTests/fast/overflow/flexbox-abspos-overflow-expected.html: Added.
* LayoutTests/fast/overflow/flexbox-abspos-overflow.html: Added.
* LayoutTests/fast/overflow/flexbox-multiline-overflow-expected.html: Added.
* LayoutTests/fast/overflow/flexbox-multiline-overflow.html: Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-flexbox/negative-overflow-002-expected.txt:
 Partial revert of 273737@main
* 
LayoutTests/platform/ios/imported/w3c/web-platform-tests/css/css-flexbox/negative-overflow-002-expected.txt:
 Partial revert of 273737@main
* Source/WebCore/rendering/RenderFlexibleBox.cpp:
(WebCore::RenderFlexibleBox::layoutBlock): Shift reset for 
m_justifyContentStartOverflow out of the flex line loop.
(WebCore::contentAlignmentStartOverflow): Fix logic to correctly account for 
reversing.
(WebCore::RenderFlexibleBox::layoutAndPlaceChildren): Update call to pass 
reversing parameter.
(WebCore::RenderFlexibleBox::alignFlexLines): Update call to pass reversing 
parameter, and fix multiline handling.

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to