Respect break-inside:avoid on table rows

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

Index: third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp b/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
index ce516459d4e6ca3b93c1933d554209b057a3bd74..06de8f2b5341a30068b7b59ffea4da611e015a02 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
@@ -1643,6 +1643,12 @@ RootInlineBox* LayoutBlockFlow::determineStartPosition(LineLayoutState& layoutSt
         LayoutUnit paginationDelta;
         for (curr = firstRootBox(); curr && !curr->isDirty(); curr = curr->nextRootBox()) {
             if (paginated) {
+                // If a cell's row wants to avoid straddling page breaks then we always need to layout all

[mstensho (USE GERRIT), 2016/04/04 21:05:39]

Pagination may indeed call for deeper layout passes, but this doesn't look like
the right place to do it. This code is only run when children are inline. What
about block children?

I think this is something that markForPaginationRelayoutIfNeeded() should be in
charge of. I was kind of hoping that it already handled this correctly, but then
there's this change right here, which kind of suggests otherwise. :)

+                // the content in the cell at once.
+                if (isTableCell() && toLayoutBox(parent())->getPaginationBreakability() != AllowAnyBreaks) {
+                    layoutState.markForFullLayout();
+                    break;
+                }
                 paginationDelta -= curr->paginationStrut();
                 adjustLinePositionForPagination(*curr, paginationDelta);
                 if (paginationDelta) {