Respect break-inside:avoid on table rows

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

 /*
  * Copyright (C) 1997 Martin Jones (mjones@kde.org)
  *           (C) 1997 Torben Weis (weis@kde.org)
  *           (C) 1998 Waldo Bastian (bastian@kde.org)
  *           (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2009, 2013 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 295 matching lines @@
     bool hasMultipleCellLevels() const { return m_hasMultipleCellLevels; }
 
     const char* name() const override { return "LayoutTableSection"; }
 
     // Whether a section has opaque background depends on many factors, e.g. border spacing,
     // border collapsing, missing cells, etc.
     // For simplicity, just conservatively assume all table sections are not opaque.
     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;
 
 private:
     LayoutObjectChildList* virtualChildren() override { return children(); }
     const LayoutObjectChildList* virtualChildren() const override { return children(); }
 
     bool isOfType(LayoutObjectType type) const override { return type == LayoutObjectTableSection || LayoutBox::isOfType(type); }
 
@@ skipping 91 matching lines @@
     // Invisible borders are never stored in this map.
     using CellsCollapsedBordersMap = HashMap<std::pair<const LayoutTableCell*, int>, CollapsedBorderValue>;
     CellsCollapsedBordersMap m_cellsCollapsedBorders;
 };
 
 DEFINE_LAYOUT_OBJECT_TYPE_CASTS(LayoutTableSection, isTableSection());
 
 } // namespace blink
 
 #endif // LayoutTableSection_h