[webkit-changes] [WebKit/WebKit] b70eda: [Multicol][IFC] Enable absolute positioned content...

Antti Koivisto noreply at github.com
Mon Sep 25 08:40:51 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b70eda7d68835d7dfa22e86463fbb90263b0e619
      https://github.com/WebKit/WebKit/commit/b70eda7d68835d7dfa22e86463fbb90263b0e619
  Author: Antti Koivisto <antti at apple.com>
  Date:   2023-09-25 (Mon, 25 Sep 2023)

  Changed paths:
    M LayoutTests/fast/multicol/content-bounding-box-rtl.html
    M Source/WebCore/layout/integration/LayoutIntegrationCoverage.cpp
    M Source/WebCore/layout/integration/LayoutIntegrationCoverage.h

  Log Message:
  -----------
  [Multicol][IFC] Enable absolute positioned content in multicolumn
https://bugs.webkit.org/show_bug.cgi?id=262031
rdar://problem/115982927

Reviewed by Alan Baradlay.

While ideally absolute positioned object with static position would stick to its line when paginated
(it does in Chrome) we don't implement this in legacy either. As a result there is nothing to here.

* LayoutTests/fast/multicol/content-bounding-box-rtl.html:

The new expected result here is logically more correct (or equally wrong in practice) compared to legacy.

* Source/WebCore/layout/integration/LayoutIntegrationCoverage.cpp:
(WebCore::LayoutIntegration::printReason):
(WebCore::LayoutIntegration::canUseForChild):
(WebCore::LayoutIntegration::canUseForLineLayoutWithReason):
* Source/WebCore/layout/integration/LayoutIntegrationCoverage.h:

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




More information about the webkit-changes mailing list