[webkit-changes] [WebKit/WebKit] 485c3b: [IFC] Move empty line handling with spanning inlin...

[Alan Baradlay]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 485c3b2ed968298037bbff1961530baeace01d49
      https://github.com/WebKit/WebKit/commit/485c3b2ed968298037bbff1961530baeace01d49
  Author: Alan Baradlay <zalan at apple.com>
  Date:   2023-01-21 (Sat, 21 Jan 2023)

  Changed paths:
    M Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.cpp
    M Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.h
    M Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayLineBuilder.cpp

  Log Message:
  -----------
  [IFC] Move empty line handling with spanning inline to InlineDisplayContentBuilder::appendSpanningInlineBoxDisplayBox
https://bugs.webkit.org/show_bug.cgi?id=250940

Reviewed by Antti Koivisto.

Move "empty line' logic to appendSpanningInlineBoxDisplayBox so that bidi content can take advantage of it too.

* Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.cpp:
(WebCore::Layout::InlineDisplayContentBuilder::appendInlineBoxDisplayBox): No need to reconsider it. Empty lines need no empty display boxes.
(WebCore::Layout::InlineDisplayContentBuilder::appendSpanningInlineBoxDisplayBox):
(WebCore::Layout::InlineDisplayContentBuilder::processNonBidiContent):
* Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.h:
* Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayLineBuilder.cpp: Figured out.
(WebCore::Layout::InlineDisplayLineBuilder::build const):

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