Bug 255854

Summary:	[css-tables] Percentage sizing of table cell replaced children with scrollbar
Product:	WebKit	Reporter:	Ahmad Saleem <ahmad.saleem792>
Component:	Tables	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	brandon, webkit-bug-importer, zalan
Priority:	P2	Keywords:	BrowserCompat, InRadar, WPTImpact
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

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.