[Webkit-unassigned] [Bug 92800] Table test is invalidating the wrong rect

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jul 31 17:04:29 PDT 2012


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





--- Comment #12 from Eric Seidel <eric at webkit.org>  2012-07-31 17:04:30 PST ---
This is the line where the cell gets marked for relayout:

            if (intrinsicPaddingBefore != oldIntrinsicPaddingBefore || intrinsicPaddingAfter != oldIntrinsicPaddingAfter)
                cell->setNeedsLayout(true, MarkOnlyThis);

inside RenderTableSection::layoutRows().

In this case, the new intrinsic padding is 148 (in layout units), and was previous 0.  I'm not entirely sure what that means.


I guess this brings us back to: Why do Table Cells need to layout twice?  Can we kill the first layout?

Maybe we should have TableCells check their parent table for their "checkForRepaintDuringLayout()" status.  They'd have to propagate this status down into their children, which could get awkward.

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



More information about the webkit-unassigned mailing list