Respect break-inside:avoid on table rows

third_party/WebKit/Source/core/layout/LayoutTableRow.h

Index: third_party/WebKit/Source/core/layout/LayoutTableRow.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutTableRow.h b/third_party/WebKit/Source/core/layout/LayoutTableRow.h
index 753da7577b956e629224c10e02e2b53de07350d3..727fb93231c6e355d4edf3ec8ae5af35dab4689b 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTableRow.h
+++ b/third_party/WebKit/Source/core/layout/LayoutTableRow.h
@@ -128,6 +128,8 @@ public:
 
     bool backgroundIsKnownToBeOpaqueInRect(const LayoutRect&) const override;
 
+    LayoutUnit paginationStrutPropagatedFromCell() const { return m_paginationStrutPropagatedFromCell; }
+    void setPaginationStrutPropagatedFromCell(LayoutUnit strut) { m_paginationStrutPropagatedFromCell = strut; };

[mstensho (USE GERRIT), 2016/02/03 14:38:48]

What if the cells don't all have the same strut?

Also: no need for the trailing semi-colon. And I think we generally like to have
a blank line before "private:".

[rhogan, 2016/02/24 18:58:17]

I can't come up with a test in which they don't - which is how I would expect a
row to work - always expanding cells to the height of the tallest one. I've
added an ASSERT.

 private:
     LayoutObjectChildList* virtualChildren() override { return children(); }
     const LayoutObjectChildList* virtualChildren() const override { return children(); }
@@ -165,6 +167,7 @@ private:
     // rowIndex() above instead. This is to ensure that we never read this
     // value before it is set.
     unsigned m_rowIndex : 31;
+    LayoutUnit m_paginationStrutPropagatedFromCell;

[mstensho (USE GERRIT), 2016/02/03 14:38:48]

Not sure if LayoutUnit should be used in table code. It's mostly about ints
there.

 };
 
 DEFINE_LAYOUT_OBJECT_TYPE_CASTS(LayoutTableRow, isTableRow());