Bug 251178 - Whitespace incorrectly suppressed inside table.
Summary: Whitespace incorrectly suppressed inside table.
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: http://wpt.live/css/css-tables/whites...
Keywords: BrowserCompat, InRadar, WPTImpact
Depends on:
Blocks:
 
Reported: 2023-01-25 15:11 PST by Emilio Cobos Álvarez (:emilio)
Modified: 2023-06-21 16:20 PDT (History)
6 users (show)

See Also:

Attachments
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 Emilio Cobos Álvarez (:emilio) 2023-01-25 15:11:09 PST 
In https://bug1811353.bmoattachments.org/attachment.cgi?id=9314059, Blink and WebKit render tables inconsistently from Gecko and the spec.

See also https://github.com/w3c/csswg-drafts/issues/8358 and https://bugs.chromium.org/p/chromium/issues/detail?id=1410345 for context, spec references, and equivalent Chromium bug.
Comment 2 Emilio Cobos Álvarez (:emilio) 2023-01-25 18:20:26 PST 
Yep that looks exactly like it.
Comment 3 Radar WebKit Bug Importer 2023-02-01 15:12:25 PST 
<rdar://problem/104934417>
Comment 4 Ahmad Saleem 2023-03-09 05:12:47 PST 
(In reply to Karl Dubost from comment #1)
> would it be
> https://searchfox.org/wubkat/rev/1c8b29ff201022e3a20578244e85d25d1b7706c4/
> Source/WebCore/rendering/updating/RenderTreeUpdater.cpp#429-431

I tested locally by removing just "parentRenderer.isTable()" make us pass "WPT" test, if we remove other bits, it makes tables go haywire.

If you want to do PR, you can do so or if you want me to do, I am happy to do it as well. Thanks!
Comment 5 Karl Dubost 2023-03-09 07:24:21 PST 
Ahmad, feel free to give it a go.