Move below headers even when row doesn't require a strut

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 354654ba43718745c260302d2d579188df3eef80..b0835dde30d618f5abc2ff6cddeecf02fd2a461a 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTableSection.h
+++ b/third_party/WebKit/Source/core/layout/LayoutTableSection.h
@@ -26,6 +26,7 @@
 #define LayoutTableSection_h
 
 #include "core/CoreExport.h"
+#include "core/layout/LayoutState.h"
 #include "core/layout/LayoutTable.h"
 #include "core/layout/LayoutTableBoxComponent.h"
 #include "wtf/Vector.h"
@@ -312,6 +313,13 @@ class CORE_EXPORT LayoutTableSection final : public LayoutTableBoxComponent {
   }
 
   int paginationStrutForRow(LayoutTableRow*, LayoutUnit logicalOffset) const;
+  bool needsStrutForTableHeaders(const LayoutState& state,

[mstensho (USE GERRIT), 2016/10/04 21:02:38]

I think I'd non-inline this, and skip the LayoutState parameter.

+                                 LayoutUnit logicalOffset) {
+    return state.heightOffsetForTableHeaders() &&
+           (pageRemainingLogicalHeightForOffset(

[mstensho (USE GERRIT), 2016/10/04 21:02:38]

Expression doesn't need to be in parentheses.

(Man, this 80 cols limit makes code hard to read sometimes) :(

[rhogan, 2016/10/04 21:10:45]

God it's awful.

+                logicalOffset, LayoutBlock::AssociateWithLatterPage) ==

[mstensho (USE GERRIT), 2016/10/04 21:02:38]

AssociateWithLatterPage is defined in LayoutBox, so no need to prefix it with a
scope.

+            pageLogicalHeightForOffset(logicalOffset));
+  };
 
   void setOffsetForRepeatingHeader(LayoutUnit offset) {
     m_offsetForRepeatingHeader = offset;