Bug 248487 - Remove a float from an element's list even if its style suggests it can't contain floats
Summary: Remove a float from an element's list even if its style suggests it can't con...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-11-29 12:45 PST by Ahmad Saleem
Modified: 2022-12-05 06:42 PST (History)
4 users (show)

See Also:

Attachments
Visual Glitch Video on JSFiddle Test Case (9.00 MB, video/quicktime)
2022-12-01 08:06 PST, Ahmad Saleem 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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>