Bug 247513 - Strip anonymous blocks when change in style removes need for them
Summary: Strip anonymous blocks when change in style removes need for them
Status: RESOLVED INVALID
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-04 17:46 PDT by Ahmad Saleem
Modified: 2023-03-10 18:23 PST (History)
4 users (show)

See Also:


Attachments

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