Respect break-inside:avoid on table rows

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

Index: third_party/WebKit/Source/core/layout/LayoutTableSection.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutTableSection.h b/third_party/WebKit/Source/core/layout/LayoutTableSection.h
index bc28e972d5aba743c58880c43cb1d670e0297e5c..c19edadee4d3e7b5ffa1b15ab6ddd03d70d4daec 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTableSection.h
+++ b/third_party/WebKit/Source/core/layout/LayoutTableSection.h
@@ -313,6 +313,8 @@ public:
     bool foregroundIsKnownToBeOpaqueInRect(const LayoutRect&, unsigned) const override { return false; }
     bool backgroundIsKnownToBeOpaqueInRect(const LayoutRect&) const override { return false; }
 
+    int getPaginationStrutForRow(LayoutTableRow*, LayoutUnit logicalOffset) const;

[mstensho (USE GERRIT), 2016/03/29 09:57:46]

s/getP/p/ ?

[rhogan, 2016/03/29 19:04:37]

Is that the new style? I still see getPaginationBreakability everywhere else.

[mstensho (USE GERRIT), 2016/03/31 09:55:04]

I don't think the intention is to prefix all getters with "get". There's just
some rule in the Google / Chromium style guide that says that names that only
differ in uppercase vs. lowercase or with or without underscore is verboten. So
it cannot be PaginationBreakability paginationBreakability(), because
strcasecmp("PaginationBreakability", "paginationBreakability") == 0.

+
 protected:
     void styleDidChange(StyleDifference, const ComputedStyle* oldStyle) override;
     bool nodeAtPoint(HitTestResult&, const HitTestLocation& locationInContainer, const LayoutPoint& accumulatedOffset, HitTestAction) override;