[webkit-changes] [WebKit/WebKit] b741d4: (REGRESSION: 267587 at main) Text indent is incorrect...

Alan Baradlay noreply at github.com
Thu Nov 23 06:49:58 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b741d4c7aa5666a19755bab586c7e6baa06f02a4
      https://github.com/WebKit/WebKit/commit/b741d4c7aa5666a19755bab586c7e6baa06f02a4
  Author: Alan Baradlay <zalan at apple.com>
  Date:   2023-11-23 (Thu, 23 Nov 2023)

  Changed paths:
    A LayoutTests/fast/text/text-indent-with-intrusive-float-expected.html
    A LayoutTests/fast/text/text-indent-with-intrusive-float.html
    M Source/WebCore/layout/formattingContexts/inline/InlineLineBuilder.cpp

  Log Message:
  -----------
  (REGRESSION: 267587 at main) Text indent is incorrect when paragraph is affected by a floating element
https://bugs.webkit.org/show_bug.cgi?id=265216
<rdar://problem/118701491>

Reviewed by Antti Koivisto.

Before 267587 at main (which merged all the float constraints codepaths), we ignored the text-indent during the _initial_ constraint computation
and adjusted the line rect later.
At 267587 at main, we started adding the text-indent value twice to the line rect; first while computing the initial constraint and
the second time when we would normally add it.

* LayoutTests/fast/text/text-indent-with-intrusive-float-expected.html: Added.
* LayoutTests/fast/text/text-indent-with-intrusive-float.html: Added.
* Source/WebCore/layout/formattingContexts/inline/InlineLineBuilder.cpp:
(WebCore::Layout::LineBuilder::initialConstraintsForLine const): Let's go back to "before 267587 at main" state.
(WebCore::Layout::LineBuilder::floatConstrainedRect const): Add some comment and change variable names to reflect what they mean.

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




More information about the webkit-changes mailing list