Update our treatment of repeating headers in tables

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

 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.

[mstensho (USE GERRIT), 2017/05/22 10:40:44]

This looks like a revert of https://codereview.chromium.org/2545243002/ , at
least to some extent. Could you elaborate a little on that in the commit
message?

  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  *
@@ skipping 63 matching lines @@
   ~LayoutState();
 
   bool IsPaginated() const { return is_paginated_; }
 
   // The page logical offset is the object's offset from the top of the page in
   // the page progression direction (so an x-offset in vertical text and a
   // y-offset for horizontal text).
   LayoutUnit PageLogicalOffset(const LayoutBox&,
                                const LayoutUnit& child_logical_offset) const;
 
+  LayoutUnit HeightOffsetForTableHeaders() const {
+    return height_offset_for_table_headers_;
+  };

[mstensho (USE GERRIT), 2017/05/22 10:40:44]

Drop the semicolon here, perhaps?

[rhogan, 2017/05/22 18:06:04]

Done.

+  void SetHeightOffsetForTableHeaders(LayoutUnit offset) {
+    height_offset_for_table_headers_ = offset;
+  };

[mstensho (USE GERRIT), 2017/05/22 10:40:44]

And here

[rhogan, 2017/05/22 18:06:04]

Done.

+
   const LayoutSize& PaginationOffset() const { return pagination_offset_; }
   bool ContainingBlockLogicalWidthChanged() const {
     return containing_block_logical_width_changed_;
   }
 
   bool PaginationStateChanged() const { return pagination_state_changed_; }
   void SetPaginationStateChanged() { pagination_state_changed_ = true; }
 
   LayoutState* Next() const { return next_; }
 
@@ skipping 10 matching lines @@
   bool pagination_state_changed_ : 1;
 
   LayoutFlowThread* flow_thread_;
 
   LayoutState* next_;
 
   // x/y offset from the logical top / start of the first page. Does not include
   // relative positioning or scroll offsets.
   LayoutSize pagination_offset_;
 
+  // The height we need to make available for repeating table headers in
+  // paginated layout.
+  LayoutUnit height_offset_for_table_headers_;
+
   LayoutObject& layout_object_;
 };
 
 }  // namespace blink
 
 #endif  // LayoutState_h