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

Source/core/rendering/RenderBox.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  *           (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 4125 matching lines @@
 
     // Only auto width objects can possibly shrink to avoid floats.
     return style()->width().isAuto();
 }
 
 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())
         return;
 
     bool isFlipped = style()->isFlippedBlocksWritingMode();
     bool isHorizontal = isHorizontalWritingMode();
 
     LayoutRect borderBox = borderBoxRect();
     LayoutUnit overflowMinX = borderBox.x();
@@ skipping 534 matching lines @@
         return 0;
 
     if (!layoutState && !flowThreadContainingBlock())
         return 0;
 
     RenderBlock* containerBlock = containingBlock();
     return containerBlock->offsetFromLogicalTopOfFirstPage() + logicalTop();
 }
 
 } // namespace WebCore