252141 – Fixed Layout excess width distribution in table

Bug 252141 - Fixed Layout excess width distribution in table

Summary: Fixed Layout excess width distribution in table

Status:	NEW

Alias:	None

Product:	WebKit
Classification:	Unclassified
Component:	Tables (show other bugs)
Version:	WebKit Nightly Build
Hardware:	Unspecified Unspecified

Importance:	P2 Normal
Assignee:	Nobody

URL:	https://wpt.fyi/results/css/css-table...
Keywords:	BrowserCompat, InRadar, WPTImpact

Depends on:
Blocks:

Reported:	2023-02-12 18:52 PST by Karl Dubost
Modified:	2023-02-16 10:11 PST (History)
CC List:	3 users (show)

See Also:

Attachments
WebKit Changes - Compiles but not fixes this test (383.84 KB, image/png) 2023-02-16 10:11 PST, Ahmad Saleem	no flags	Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Karl Dubost 2023-02-12 18:52:31 PST

This WPT test fails
https://wpt.fyi/results/css/css-tables/fixed-layout-excess-width-distribution-001.html


It was fixed in Chromium with
https://chromium-review.googlesource.com/c/chromium/src/+/644188/5/third_party/WebKit/Source/core/layout/TableLayoutAlgorithmFixed.cpp#b225

The equivalent code in WebKit is at
https://searchfox.org/wubkat/rev/dbe54de0044f94bf3d6a7fbf07fc29ace272bb58/Source/WebCore/rendering/FixedTableLayout.cpp#239

Comment 1 Radar WebKit Bug Importer 2023-02-12 18:52:44 PST

<rdar://problem/105372108>

Comment 2 Ahmad Saleem 2023-02-16 10:11:49 PST

Created attachment 465027 [details]
WebKit Changes - Compiles but not fixes this test

I compiled local build based on the Chromium / Blink patch as shown in the image (I also did change in .h file) and it compiles but it does not passes the WPT test case.

Just wanted to highlight. If I am doing anything wrong, would appreciate help. Thanks!