Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 81e7db38bc4a035c19cfb762dd4c6d0756f598ea
      
https://github.com/WebKit/WebKit/commit/81e7db38bc4a035c19cfb762dd4c6d0756f598ea
  Author: Alan Baradlay <[email protected]>
  Date:   2024-11-18 (Mon, 18 Nov 2024)

  Changed paths:
    M Source/WebCore/layout/floats/FloatingContext.cpp
    M Source/WebCore/layout/floats/FloatingContext.h
    M Source/WebCore/layout/floats/PlacedFloats.cpp
    M Source/WebCore/layout/floats/PlacedFloats.h
    M Source/WebCore/layout/formattingContexts/FormattingGeometry.cpp
    M Source/WebCore/layout/formattingContexts/block/BlockFormattingContext.cpp

  Log Message:
  -----------
  [Cleanup][Floats] Move FloatingContext's top/bottom methods to PlacedFloats
https://bugs.webkit.org/show_bug.cgi?id=283239

Reviewed by Antti Koivisto.

Since both 'top' and 'bottom' return coords relative to BFC root, it may be 
misleading to
have these functions on the FloatingContext where the root of the 
FloatingContext may not always be
the same as the root of the BFC (i.e. BFC with nested IFCs)

* Source/WebCore/layout/floats/FloatingContext.cpp:
(WebCore::Layout::FloatingContext::positionForFloat const):
(WebCore::Layout::FloatingContext::verticalPositionWithClearance const):
(WebCore::Layout::FloatingContext::isFloatingCandidateStartPositionedInPlacedFloats
 const):
(WebCore::Layout::FloatingContext::bottom const): Deleted.
(WebCore::Layout::FloatingContext::top const): Deleted.
* Source/WebCore/layout/floats/FloatingContext.h:
(WebCore::Layout::FloatingContext::leftBottom const): Deleted.
(WebCore::Layout::FloatingContext::rightBottom const): Deleted.
(WebCore::Layout::FloatingContext::bottom const): Deleted.
* Source/WebCore/layout/floats/PlacedFloats.cpp:
(WebCore::Layout::PlacedFloats::topMost const):
(WebCore::Layout::PlacedFloats::bottomMost const):
* Source/WebCore/layout/floats/PlacedFloats.h:
* Source/WebCore/layout/formattingContexts/FormattingGeometry.cpp:
(WebCore::Layout::FormattingGeometry::complicatedCases const):
* Source/WebCore/layout/formattingContexts/block/BlockFormattingContext.cpp:
(WebCore::Layout::BlockFormattingContext::usedContentHeight const):

Canonical link: https://commits.webkit.org/286723@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