Don't propagate pagination struts to flow threads.

third_party/WebKit/Source/core/layout/LayoutBlockFlow.cpp

Index: third_party/WebKit/Source/core/layout/LayoutBlockFlow.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutBlockFlow.cpp b/third_party/WebKit/Source/core/layout/LayoutBlockFlow.cpp
index e394118edb9f5e1d1721112df0aa43ff0bff9fd4..a727657b260549e8ef317013f4a72cbbde77f3a8 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBlockFlow.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBlockFlow.cpp
@@ -2887,6 +2887,14 @@ bool LayoutBlockFlow::allowsPaginationStrut() const
     // TODO(mstensho): But we *should*.
     if (isOutOfFlowPositioned())
         return false;
+    if (isLayoutFlowThread()) {
+        // Don't let the strut escape the fragmentation context and get lost.
+        // TODO(mstensho): if we're in a nested fragmentation context, we should ideally convert

[leviw_travelin_and_unemployed, 2015/11/05 22:36:06]

No capital 'i' in 'if'? I suppose I can let that slide ;)

[mstensho (USE GERRIT), 2015/11/05 22:44:09]

That's out of the question! Fixed.

+        // and propagate the strut to the outer fragmentation context, so that the inner one is
+        // fully pushed to the next outer fragmentainer, instead of taking up unusable space in the
+        // previous one. But currently we have no mechanism in place to handle this.
+        return false;
+    }
     LayoutBlock* containingBlock = this->containingBlock();
     return containingBlock && containingBlock->isLayoutBlockFlow();
 }