Branch: refs/heads/main Home: https://github.com/WebKit/WebKit Commit: aeaddddc170f66ccf5dcf3dec867fd34623cc396 https://github.com/WebKit/WebKit/commit/aeaddddc170f66ccf5dcf3dec867fd34623cc396 Author: Alan Baradlay <za...@apple.com> Date: 2023-05-06 (Sat, 06 May 2023)
Changed paths: M Source/WebCore/layout/formattingContexts/inline/InlineFormattingContext.cpp M Source/WebCore/layout/formattingContexts/inline/InlineLineBuilder.cpp M Source/WebCore/layout/formattingContexts/inline/InlineLineBuilder.h Log Message: ----------- [IFC] Remove redundant LineBuilder constructor https://bugs.webkit.org/show_bug.cgi?id=256285 Reviewed by Antti Koivisto. Intrinsic width codepath now can use LineBuilder as is (by passing in newly created temporary states), since LineBuilder does not mutate any persistent state (box geometry) anymore. * Source/WebCore/layout/formattingContexts/inline/InlineFormattingContext.cpp: (WebCore::Layout::InlineFormattingContext::computedIntrinsicWidthForConstraint const): * Source/WebCore/layout/formattingContexts/inline/InlineLineBuilder.cpp: (WebCore::Layout::LineBuilder::LineBuilder): (WebCore::Layout::LineBuilder::floatConstraints const): (WebCore::Layout::LineBuilder::adjustLineRectForInitialLetterIfApplicable): (WebCore::Layout::LineBuilder::tryPlacingFloatBox): * Source/WebCore/layout/formattingContexts/inline/InlineLineBuilder.h: (WebCore::Layout::LineBuilder::inlineLayoutState const): (WebCore::Layout::LineBuilder::blockLayoutState const): (WebCore::Layout::LineBuilder::floatingState): (WebCore::Layout::LineBuilder::floatingState const): Canonical link: https://commits.webkit.org/263757@main _______________________________________________ webkit-changes mailing list webkit-changes@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-changes