[LFC][Floats] Add support for clear on float box

[zalan]

<div style="float: left; clear: both">

[zalan]

Created attachment 409867 [details]
Patch

[Antti Koivisto]

Comment on attachment 409867 [details]
Patch

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

> Source/WebCore/layout/floats/FloatingContext.cpp:262
> +        auto floatBottom = Optional<PositionInContextRoot> { };
> +        switch (layoutBox.style().clear()) {
> +        case Clear::Left:
> +            floatBottom = floatingState().leftBottom(root());
> +            break;
> +        case Clear::Right:
> +            floatBottom = floatingState().rightBottom(root());
> +            break;
> +        case Clear::Both:
> +            floatBottom = floatingState().bottom(root());
> +            break;
> +        default:
> +            ASSERT_NOT_REACHED();
> +        }

No lambda?

[zalan]

Created attachment 409885 [details]
Patch

[zalan]

(In reply to Antti Koivisto from comment #2)
> Comment on attachment 409867 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=409867&action=review
> 
> > Source/WebCore/layout/floats/FloatingContext.cpp:262
> > +        auto floatBottom = Optional<PositionInContextRoot> { };
> > +        switch (layoutBox.style().clear()) {
> > +        case Clear::Left:
> > +            floatBottom = floatingState().leftBottom(root());
> > +            break;
> > +        case Clear::Right:
> > +            floatBottom = floatingState().rightBottom(root());
> > +            break;
> > +        case Clear::Both:
> > +            floatBottom = floatingState().bottom(root());
> > +            break;
> > +        default:
> > +            ASSERT_NOT_REACHED();
> > +        }
> 
> No lambda?
Certainly an oversight on my part.

[EWS]

Committed r267700: <https://trac.webkit.org/changeset/267700>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 409885 [details].

[Radar WebKit Bug Importer]

<rdar://problem/69698893>