[Webkit-unassigned] [Bug 40775] Table Cell Layering

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=40775





--- Comment #9 from Fady Samuel <fsamuel at chromium.org>  2010-06-21 13:16:33 PST ---
(In reply to comment #8)
> (From update of attachment 59037 [details])
> Some initial feedback:
> 
> (3) I'm not convinced that the overlap level member variable is needed.  Adding 4 bytes to all RenderTableCells for this seems problematic (same with the extra boolean that was added).  Can't paint order be inferred from ordering in the CellStruct? Maybe the answer is no, but think about it.  I'd consider using hashes in the .cpp file though rather than bloating RenderTableCell for a rare case (e.g., like we do with column members in RenderBlock).

Everything else you said was pretty straightforward. Inferring paint order will definitely be a bit tricky but first instinct is it's possible. I'll spend a bit of time thinking of potential corner cases where this might not be possible. If it is possible, it will also be a bit tricky to do this efficiently. It's an interesting thought, and I agree, it would be nice to avoid those extra 4 bytes.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.