WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
247513
Strip anonymous blocks when change in style removes need for them
https://bugs.webkit.org/show_bug.cgi?id=247513
Summary
Strip anonymous blocks when change in style removes need for them
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
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2022-11-11 16:46:19 PST
<
rdar://problem/102261782
>
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.
Top of Page
Format For Printing
XML
Clone This Bug