Bug 247513

Summary:	Strip anonymous blocks when change in style removes need for them
Product:	WebKit	Reporter:	Ahmad Saleem <ahmad.saleem792>
Component:	Layout and Rendering	Assignee:	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

Description Ahmad Saleem 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!

Comment 1 Radar WebKit Bug Importer 2022-11-11 16:46:19 PST

<rdar://problem/102261782>

Comment 2 Ahmad Saleem 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.

Comment 3 Ahmad Saleem 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!