Bug 207099

Summary: [LFC][Integration] Support intruding floats
Product: WebKit Reporter: Antti Koivisto <koivisto>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
wip
none
patch none

Description Antti Koivisto 2020-02-02 05:14:24 PST
SLL supports intruding floats, LFC integration should support them too.
Comment 1 Antti Koivisto 2020-02-02 06:54:24 PST
Created attachment 389474 [details]
wip
Comment 2 Antti Koivisto 2020-02-09 15:56:20 PST
Created attachment 390217 [details]
patch
Comment 3 zalan 2020-02-09 16:06:31 PST
Comment on attachment 390217 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=390217&action=review

> Source/WebCore/layout/floats/FloatingState.cpp:51
> +FloatingState::FloatItem::FloatItem(Position position, Display::Box absoluteDisplayBox)
> +    : m_position(position)
> +    , m_absoluteDisplayBox(absoluteDisplayBox)

At some point this needs to be expanded to cover isDescendantOfFormattingRoot() as well (with the current integration setup/content, we don't call it).
Comment 4 WebKit Commit Bot 2020-02-09 17:51:49 PST
Comment on attachment 390217 [details]
patch

Clearing flags on attachment: 390217

Committed r256107: <https://trac.webkit.org/changeset/256107>
Comment 5 WebKit Commit Bot 2020-02-09 17:51:50 PST
All reviewed patches have been landed.  Closing bug.
Comment 6 Radar WebKit Bug Importer 2020-02-09 17:52:15 PST
<rdar://problem/59298921>