Documents with many lines overallocated pagination info.

Source/core/rendering/RootInlineBox.cpp

Index: Source/core/rendering/RootInlineBox.cpp
diff --git a/Source/core/rendering/RootInlineBox.cpp b/Source/core/rendering/RootInlineBox.cpp
index 3bcc6fd62bd62739c7a7072201df1d01d85148fe..9c6a1dc7df4d39cf92fe9a5fce7fa290feaf143c 100644
--- a/Source/core/rendering/RootInlineBox.cpp
+++ b/Source/core/rendering/RootInlineBox.cpp
@@ -249,7 +249,9 @@ LayoutUnit RootInlineBox::alignBoxesInBlockDirection(LayoutUnit heightOfBlock, G
     maxHeight = max<LayoutUnit>(0, maxHeight); // FIXME: Is this really necessary?
 
     setLineTopBottomPositions(lineTop, lineBottom, heightOfBlock, heightOfBlock + maxHeight);
-    setPaginatedLineWidth(block().availableLogicalWidthForContent());
+    bool paginated = block().view()->layoutState() && block().view()->layoutState()->isPaginated();

[leviw_travelin_and_unemployed, 2014/03/20 18:05:38]

If there's no LayoutState, this logic doesn't seem right.

[leviw_travelin_and_unemployed, 2014/03/20 18:41:26]

So thinking about this more, we should never be in this code without a
LayoutState. I've also verified, with an ASSERT, that we don't currently ever
hit this code path without one.

I think the proper fix is to just use if
(block().view()->layoutState()->isPaginated()) as your if statement.

[Daniel Bratell, 2014/03/21 10:33:16]

Done.

+    if (paginated)
+        setPaginatedLineWidth(block().availableLogicalWidthForContent());
 
     LayoutUnit annotationsAdjustment = beforeAnnotationsAdjustment();
     if (annotationsAdjustment) {