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 | ||
Ahmad Saleem
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 | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/102261782>
Ahmad Saleem
(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
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!