Negative row height when cell has percentage height.

Source/core/layout/LayoutTableSection.cpp

Index: Source/core/layout/LayoutTableSection.cpp
diff --git a/Source/core/layout/LayoutTableSection.cpp b/Source/core/layout/LayoutTableSection.cpp
index 29e2a0c609103400515219cc23d2224ab18a3e07..fd0ab895e18cc5d7560256a761aac38aeec0eac0 100644
--- a/Source/core/layout/LayoutTableSection.cpp
+++ b/Source/core/layout/LayoutTableSection.cpp
@@ -313,6 +313,7 @@ void LayoutTableSection::distributeExtraRowSpanHeightToPercentRows(LayoutTableCe
                 // FIXME: Note that this is wrong if we have a percentage above 100% and may make us grow
                 // above the available space.
 
+                toAdd = std::max(toAdd, 0);

[esprehn, 2015/03/26 08:23:11]

How does toAdd become negative here?

[a.suchit2, 2015/03/26 09:38:35]

If percent height is less than the rowHeight based on the content in the cells
of the row then percent height should be ignored.

just consider : 
rowsHeight[row - rowIndex] -> 10 px => this is the row height based on content
on the cells of this row.

Total table height is 20 px

then 1% (percent height for the row) of 20 px is 2.

toAdd become 2 - 20 = -18

[Julien - ping for review, 2015/03/26 15:22:10]

It's not the only place where we do something similar and it's reasonable to do
so here IMO.

                 toAdd = std::min(toAdd, extraRowSpanningHeight);
                 accumulatedPositionIncrease += toAdd;
                 extraRowSpanningHeight -= toAdd;