Make sure table cells are laid out again when fragmentainer height changes.

Source/core/rendering/RenderBox.cpp

Index: Source/core/rendering/RenderBox.cpp
diff --git a/Source/core/rendering/RenderBox.cpp b/Source/core/rendering/RenderBox.cpp
index 63da47dd6c38ae0151e61c9aaa53724cb3e6ac3a..20df2f32461cf0c45df124a66550caa9a31c68bb 100644
--- a/Source/core/rendering/RenderBox.cpp
+++ b/Source/core/rendering/RenderBox.cpp
@@ -4143,6 +4143,18 @@ bool RenderBox::avoidsFloats() const
     return isReplaced() || hasOverflowClip() || isHR() || isLegend() || isWritingModeRoot() || isFlexItemIncludingDeprecated();
 }
 
+void RenderBox::markForPaginationRelayoutIfNeeded(SubtreeLayoutScope& layoutScope)

[Julien - ping for review, 2014/03/12 15:58:23]

Ideally this should be only used for table parts but we don't have a
RenderTablePart for them.

[mstensho (USE GERRIT), 2014/03/13 08:26:48]

We used to only do this for blocks (well.. RenderBlock, so you'd get table
boxes, flexboxes, grids and probably other things for free, in addition to
regular block containers).

But you need this for anything with children that can be paginated. Is that
really only table parts, in addition to block-ish boxes?

+{
+    ASSERT(!needsLayout());
+    if (needsLayout())

[Julien - ping for review, 2014/03/12 15:58:23]

This is a bad pattern. Either we know the ASSERT is never reached and the if is
useless or we just accept the fact that it's not enforced and remove the ASSERT.
Here it seems to be the option 1.

[mstensho (USE GERRIT), 2014/03/13 08:26:48]

Done. Indeed. I stole this code, but bad indeed. :)

+        return;
+
+    // If fragmentation height has changed, we need to lay out. No need to enter the renderer if it
+    // is childless, though.
+    if (view()->layoutState()->pageLogicalHeightChanged() && firstChild())
+        layoutScope.setChildNeedsLayout(this);
+}
+
 void RenderBox::addVisualEffectOverflow()
 {
     if (!style()->boxShadow() && !style()->hasBorderImageOutsets())