NEW223099
Out-of-flow positioned box should generate an out-of-flow fragment container 
https://bugs.webkit.org/show_bug.cgi?id=223099
Summary Out-of-flow positioned box should generate an out-of-flow fragment container
alan
Reported 2021-03-11 16:04:20 PST
<rdar://73706973>
Attachments
Patch (4.24 KB, patch)
2021-03-11 16:33 PST, alan 		no flags
DetailsFormatted DiffDiff
Patch (4.23 KB, patch)
2021-04-28 10:10 PDT, alan 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
alan
Comment 1 2021-03-11 16:33:44 PST
Created attachment 422990 [details] Patch
alan
Comment 2 2021-03-11 19:59:01 PST
Interestingly the failed tests produce the same geometries before/after. It probably needs some RenderLayer adjustment.
alan
Comment 3 2021-04-28 10:10:30 PDT
Created attachment 427275 [details] Patch
Ahmad Saleem
Comment 4 2023-10-29 07:34:44 PDT
We don't have following applied: https://searchfox.org/wubkat/source/Source/WebCore/rendering/updating/RenderTreeBuilderMultiColumn.cpp#164 auto newFragmentedFlow = WebCore::createRenderer<RenderMultiColumnFlow>(flow.document(), RenderStyle::createAnonymousStyleWithDisplay(flow.style(), DisplayType::Block)); newFragmentedFlow->initializeStyle(); auto& fragmentedFlow = *newFragmentedFlow; m_builder.blockBuilder().attach(flow, WTFMove(newFragmentedFlow), nullptr); // Reparent children preceding the fragmented flow into the fragmented flow. m_builder.moveChildren(flow, fragmentedFlow, flow.firstChild(), &fragmentedFlow, RenderTreeBuilder::NormalizeAfterInsertion::Yes);
Ahmad Saleem
Comment 5 2023-10-29 07:40:18 PDT
(In reply to Ahmad Saleem from comment #4) > We don't have following applied: > > https://searchfox.org/wubkat/source/Source/WebCore/rendering/updating/ > RenderTreeBuilderMultiColumn.cpp#164 > > auto newFragmentedFlow = > WebCore::createRenderer<RenderMultiColumnFlow>(flow.document(), > RenderStyle::createAnonymousStyleWithDisplay(flow.style(), > DisplayType::Block)); > newFragmentedFlow->initializeStyle(); > auto& fragmentedFlow = *newFragmentedFlow; > m_builder.blockBuilder().attach(flow, WTFMove(newFragmentedFlow), > nullptr); > > // Reparent children preceding the fragmented flow into the fragmented > flow. > m_builder.moveChildren(flow, fragmentedFlow, flow.firstChild(), > &fragmentedFlow, RenderTreeBuilder::NormalizeAfterInsertion::Yes); Just to update - it compiles based on above patch on WebKit ToT. :-)
Note You need to log in before you can comment on or make changes to this bug.