Bug 248487

Summary: Remove a float from an element's list even if its style suggests it can't contain floats
Product: WebKit Reporter: Ahmad Saleem <ahmad.saleem792>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Visual Glitch Video on JSFiddle Test Case none

Description Ahmad Saleem 2022-11-29 12:45:57 PST 
Hi Team,

While going through Blink's commit, I came across another potential case to make Layout & Rendering bit more robust.

Test Case (Showing "black" scrollbar on left side in Safari till repaint then it shows on left) - https://jsfiddle.net/eso3k7vw/show

Blink Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=190902

Webkit GitHub Source - https://github.com/WebKit/WebKit/blob/52128324756a99e79952e4d3431bff7fdceb9bb9/Source/WebCore/rendering/RenderBlockFlow.cpp#LL2816

Just wanted to raise bug before trying to do PR.

Thanks!
Comment 1 Ahmad Saleem 2022-12-01 08:06:43 PST 
Created attachment 463829 [details]
Visual Glitch Video on JSFiddle Test Case

@Alan - visual glitch with this test case and why I am trying to merge this because it causes Safari to do painting issue and also show scrollbar on incorrect side.
Comment 2 EWS 2022-12-05 06:41:24 PST 
Committed 257370@main (6112c5929ca9): <https://commits.webkit.org/257370@main>

Reviewed commits have been landed. Closing PR #6933 and removing active labels.
Comment 3 Radar WebKit Bug Importer 2022-12-05 06:42:22 PST 
<rdar://problem/102980611>