[webkit-changes] [WebKit/WebKit] f1664d: REGRESSION(267944 at main): wpt /css/CSS2/positioning...

Alan Baradlay noreply at github.com
Wed Nov 1 11:57:01 PDT 2023


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

  Changed paths:
    A LayoutTests/fast/block/right-to-left-out-of-flow-with-inflow-content-expected.html
    A LayoutTests/fast/block/right-to-left-out-of-flow-with-inflow-content.html
    M Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.cpp
    M Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.h

  Log Message:
  -----------
  REGRESSION(267944 at main): wpt /css/CSS2/positioning/abspos-block-level-001.html
https://bugs.webkit.org/show_bug.cgi?id=263892

Reviewed by Antti Koivisto.

This patch extends the logic of "is there an inflow content before this out of flow box" by
taking inline direction into account.

We already handle bidi (logical position vs. visual position) just fine but there's another layer of visual order here.
When the content is RTL (again not to be confused with bidi) the logic of finding out if an out of flow box
has preceding in flow content needs to start from the (visual) right edge and follow the inline direction.
(i.e. in addition to resolving the logical -> visual order, we also need to travers these run lists in inline direction order)

* Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.cpp:
(WebCore::Layout::InlineDisplayContentBuilder::processBidiContent):
(WebCore::Layout::setGeometryForOutOfFlowBoxes):
(WebCore::Layout::InlineDisplayContentBuilder::setGeometryForBlockLevelOutOfFlowBoxes):
* Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.h:
(WebCore::Layout::InlineDisplayContentBuilder::setGeometryForBlockLevelOutOfFlowBoxes):

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




More information about the webkit-changes mailing list