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 d429db7252bc7090b4ffeff46ca27de4cfc9b792..5c2af5d936590e546a8b73c6baca0fe842d2b205 100644
--- a/Source/core/layout/LayoutTableSection.cpp
+++ b/Source/core/layout/LayoutTableSection.cpp
@@ -322,6 +322,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);
                 toAdd = std::min(toAdd, extraRowSpanningHeight);

[Julien - ping for review, 2015/07/14 17:02:41]

I think a single line would be better as it would show the relationship we want:

toAdd = std::max(std::min(toAdd, extraRowSpanningHeight), 0);

Also I would do it *after* applying the min as it would ensure that a
potentially negative extraRowSpanHeight won't make toAdd negative.

[a.suchit, 2015/07/20 06:54:09]

Done.

                 accumulatedPositionIncrease += toAdd;
                 extraRowSpanningHeight -= toAdd;