Bug 247513

Summary: Strip anonymous blocks when change in style removes need for them
Product: WebKit Reporter: Ahmad Saleem <ahmad.saleem792>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: bfulgham, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Unspecified   
OS: Unspecified   

Ahmad Saleem
Reported 2022-11-04 17:46:03 PDT
Hi Team, While going through Blink commits, I came across following: Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=174420 It could be good to merge this since we are failing this test case below: Test Case - https://jsfiddle.net/L0jxg97r/show Only Chrome Canary 109 is passing it and Safari is still failing it in STP 156 while Firefox Nightly 108 also fail it. Just wanted to create bug before trying to merge it and see if I can make it work or if not then at least, we have failing test case and bug related to it. Thanks!
Attachments
Radar WebKit Bug Importer
Comment 1 2022-11-11 16:46:19 PST
Ahmad Saleem
Comment 2 2022-11-22 12:40:26 PST
(In reply to Ahmad Saleem from comment #0) > Hi Team, > > While going through Blink commits, I came across following: > > Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=174420 > > It could be good to merge this since we are failing this test case below: > > Test Case - https://jsfiddle.net/L0jxg97r/show > > Only Chrome Canary 109 is passing it and Safari is still failing it in STP > 156 while Firefox Nightly 108 also fail it. > > Just wanted to create bug before trying to merge it and see if I can make it > work or if not then at least, we have failing test case and bug related to > it. > > Thanks! Merge this - https://src.chromium.org/viewvc/blink?view=revision&revision=185200 Other one got reverted.
Ahmad Saleem
Comment 3 2023-03-10 18:23:42 PST
It seems WebKit ToT is passing this test and I also check all other tests from the commit and Safari Technology Preview 165 is also matching in them with Chrome Canary 113. Marking this as "RESOLVED INVALID". Thanks!
Note You need to log in before you can comment on or make changes to this bug.