Bug 254183 – Let siblings layout if an adjacent float may no longer affect them

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Ahmad Saleem]

Hi Team,

While going through Blink's commit, I came across another potential merge and bug fix.

Blink Commit - https://chromium.googlesource.com/chromium/src.git/+/789b5e528fb3bd9ca44341de46547c45355652ab

Test Case - https://jsfiddle.net/kqj93hp5/show

^ Failing in Safari Technology Preview 165, passes after adding following in WebKit Source:

        fb = std::max(fb, childBlockFlow->lowestFloatLogicalBottom());

In WebKit Source after this as L969:

https://searchfox.org/wubkat/source/Source/WebCore/rendering/RenderBlockFlow.cpp#968

Just wanted to raise so we can fix it.

Thanks!

[Ahmad Saleem]

(In reply to Ahmad Saleem from comment #0)
> Hi Team,
> 
> While going through Blink's commit, I came across another potential merge
> and bug fix.
> 
> Blink Commit -
> https://chromium.googlesource.com/chromium/src.git/+/
> 789b5e528fb3bd9ca44341de46547c45355652ab
> 
> Test Case - https://jsfiddle.net/kqj93hp5/show
> 
> ^ Failing in Safari Technology Preview 165, passes after adding following in
> WebKit Source:
> 
>         fb = std::max(fb, childBlockFlow->lowestFloatLogicalBottom());
> 
> In WebKit Source after this as L969:
> 
> https://searchfox.org/wubkat/source/Source/WebCore/rendering/RenderBlockFlow.
> cpp#968
> 
> Just wanted to raise so we can fix it.
> 
> Thanks!

Merging this as it is, led to crashes in my local build (It happened when I try to looked into old bugs with 'negative margin' reference to see, if they are now fixed with this or not.

Additionally, then I took the patch reference to search whether there is any follow-up on Chromium / Blink and noticed that it had regression in performance test and got superseded by this:

https://chromium.googlesource.com/chromium/src.git/+/4a961b8eca27af4108358a30753197754316524b

[Radar WebKit Bug Importer]

<rdar://problem/107283741>

[zalan]

Created attachment 465729 [details]
[fast-cq]Patch

[zalan]

Different approach as I am not convinced that the blink patch is correct.

[EWS]

Committed 262481@main (5673db1ac5ba): <https://commits.webkit.org/262481@main>

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