Bug 3235 - CSS2: Table cell coalescing in CSS broken
: CSS2: Table cell coalescing in CSS broken
Status: RESOLVED FIXED
: WebKit
Tables
: 412
: All All
: P2 Normal
Assigned To:
:
: InRadar, ReviewedForRadar
:
:
  Show dependency treegraph
 
Reported: 2005-06-01 14:56 PST by
Modified: 2009-04-21 08:49 PST (History)


Attachments
test case as attachement (216 bytes, text/html)
2005-12-28 02:12 PST, Eric Seidel
no flags Details
Patch to add test (2.15 KB, text/plain)
2009-04-21 02:00 PST, Cameron Zwarich (cpst)
darin: review+
Details


Note

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


Description From 2005-06-01 14:56:31 PST
<html>
<head>
<style>
.entries { display: table-cell; }
.sidebar { display: table-cell; }
</style>
</head>
<body>
<div class="nav"></div>
<div class="entries">One</div>
<div class="sidebar">Two</div>
</body>
</html>

"One" and "Two" should be in the same table, so they should be in a row together.  Instead they end up 
in separate tables all because of the presence of the extra "nav" div.
------- Comment #1 From 2005-06-01 14:57:37 PST -------
Apple Bug: rdar://3299883/
------- Comment #2 From 2005-12-28 02:12:04 PST -------
Created an attachment (id=5331) [details]
test case as attachement
------- Comment #3 From 2005-12-28 02:13:00 PST -------
CC'ing beth as this is in her area of expertise.
------- Comment #4 From 2008-05-31 01:06:42 PST -------
After looking into the bug I cannot seem to locate the logic for testing adjacent cells. It does not occur in all the Rendering files and my next guess was the CSSParser, which also does not appear to have the logic. Also the use of setStyle only occurs once for TABLE_CELL, and that also does not appear to contain the logic. Any clues?
------- Comment #5 From 2008-06-01 23:50:52 PST -------
Hi Bradley,

(In reply to comment #4)
> After looking into the bug I cannot seem to locate the logic for testing
> adjacent cells. It does not occur in all the Rendering files and my next guess
> was the CSSParser, which also does not appear to have the logic. Also the use
> of setStyle only occurs once for TABLE_CELL, and that also does not appear to
> contain the logic. Any clues?

I noticed that the render tree dumps are different if you position the non table cell div first or last. In the last case two RenderTableCell's are created, the first case just one. It may be worth debugging on what basis
and where these RenderTableCell's are created, maybe by stepping through the code or adding printf statements.
Cheers,

Rob.
------- Comment #6 From 2009-04-21 02:00:54 PST -------
Created an attachment (id=29644) [details]
Patch to add test

The bug is now fixed, but we should add this as a test.
------- Comment #7 From 2009-04-21 08:49:21 PST -------
Landed in r42711.