Floated <br> should not trigger a line break

[Simon Fraser (smfr)]

https://github.com/w3c/csswg-drafts/issues/5749

[Radar WebKit Bug Importer]

<rdar://problem/83154050>

[Ahmad Saleem]

Safari / WebKit ToT matches Chrome Canary 118 while Firefox Nightly 116 differs from both.

Took examples of @Simon's post from GitHub and changed into JSFiddle and linked.

[zalan]

This is pretty much about removing the following lines: (now that the spec is clear)
            clonedStyle.setDisplay(DisplayType::Inline);
            clonedStyle.setFloating(Float::None);
            clonedStyle.setPosition(PositionType::Static);
in TreeBuilder::createLayoutBox.

will fix it soon.

[Ahmad Saleem]

(In reply to zalan from comment #3)
> This is pretty much about removing the following lines: (now that the spec
> is clear)
>             clonedStyle.setDisplay(DisplayType::Inline);
>             clonedStyle.setFloating(Float::None);
>             clonedStyle.setPosition(PositionType::Static);
> in TreeBuilder::createLayoutBox.
> 
> will fix it soon.

Deleting these three lines didn't fix it on local for me. :-(

[zalan]

repurpose it for the float case. out-of-flow case is here 259631

[zalan]

bug 259631

[zalan]

(In reply to Ahmad Saleem from comment #4)
> (In reply to zalan from comment #3)
> > This is pretty much about removing the following lines: (now that the spec
> > is clear)
> >             clonedStyle.setDisplay(DisplayType::Inline);
> >             clonedStyle.setFloating(Float::None);
> >             clonedStyle.setPosition(PositionType::Static);
> > in TreeBuilder::createLayoutBox.
> > 
> > will fix it soon.
> 
> Deleting these three lines didn't fix it on local for me. :-(
yeah it's slightly more involved.