Bug 255854 - [css-tables] Percentage sizing of table cell replaced children with scrollbar
Summary: [css-tables] Percentage sizing of table cell replaced children with scrollbar
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tables (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BrowserCompat, InRadar, WPTImpact
Depends on:
Blocks:
 
Reported: 2023-04-24 00:19 PDT by Ahmad Saleem
Modified: 2023-05-18 00:53 PDT (History)
3 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 2023-04-24 00:19:55 PDT
Hi Team,

Came across another failing test case in WebKit and WPT testcase:

WPT Test Case - http://wpt.live/css/css-tables/height-distribution/percentage-sizing-of-table-cell-replaced-children-001.html

Blink Commit - https://chromium.googlesource.com/chromium/src.git/+/bf802aa687ff50a9f4017376cf92da49915f6f8b

WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/rendering/RenderBox.cpp#3764

adding '- scrollbarLogicalHeight();' fixes this locally.

Will do PR to fix it.

Thanks!
Comment 1 EWS 2023-04-24 10:15:15 PDT
Committed 263318@main (f1b46a85d183): <https://commits.webkit.org/263318@main>

Reviewed commits have been landed. Closing PR #13091 and removing active labels.
Comment 2 Radar WebKit Bug Importer 2023-04-24 10:16:21 PDT
<rdar://problem/108459503>
Comment 3 Brandon McConnell 2023-05-17 19:16:49 PDT
@Ahmad could this be related? It’s a similar bug in a similar context?
https://bugs.webkit.org/show_bug.cgi?id=245496
Comment 4 Ahmad Saleem 2023-05-18 00:53:16 PDT
(In reply to Brandon McConnell from comment #3)
> @Ahmad could this be related? It’s a similar bug in a similar context?
> https://bugs.webkit.org/show_bug.cgi?id=245496

Unfortunately, I think it isn't because I tried the test case:

https://codepen.io/brandonmcconnell/full/poZxVyR/eed0f1933bced0e2978d6a135e72ecc8

and

https://play.tailwindcss.com/tjDRHWiftz

but it is same in Safari Technology Preview 170 and Safari 16.4.1 and different compared to Chrome Canary 115.