Don't establish LayoutState for LayoutTableRow objects.

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

 /*
  * 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, 2008, 2009, 2010, 2013 Apple Inc.
  *               All rights reserved.
  * Copyright (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  *
@@ skipping 820 matching lines @@
 int LayoutTableSection::calcRowLogicalHeight() {
 #if ENABLE(ASSERT)
   SetLayoutNeededForbiddenScope layoutForbiddenScope(*this);
 #endif
 
   ASSERT(!needsLayout());
 
   LayoutTableCell* cell;
 
   // We may have to forcefully lay out cells here, in which case we need a
-  // layout state. Technically, we should also push state for the row, but since
-  // rows don't push a coordinate transform, that's not necessary.
-  LayoutState state(*this, locationOffset());
+  // layout state.
+  LayoutState state(*this);
 
   m_rowPos.resize(m_grid.size() + 1);
 
   // We ignore the border-spacing on any non-top section as it is already
   // included in the previous section's last row position.
   if (this == table()->topSection())
     m_rowPos[0] = table()->vBorderSpacing();
   else
     m_rowPos[0] = 0;
 
@@ skipping 99 matching lines @@
   ASSERT(needsLayout());
   LayoutAnalyzer::Scope analyzer(*this);
   RELEASE_ASSERT(!needsCellRecalc());
   ASSERT(!table()->needsSectionRecalc());
 
   // addChild may over-grow m_grid but we don't want to throw away the memory
   // too early as addChild can be called in a loop (e.g during parsing). Doing
   // it now ensures we have a stable-enough structure.
   m_grid.shrinkToFit();
 
-  LayoutState state(*this, locationOffset());
+  LayoutState state(*this);
 
   const Vector<int>& columnPos = table()->effectiveColumnPositions();
   LayoutUnit rowLogicalTop;
 
   SubtreeLayoutScope layouter(*this);
   for (unsigned r = 0; r < m_grid.size(); ++r) {
     Row& row = m_grid[r].row;
     unsigned cols = row.size();
     // First, propagate our table layout's information to the cells. This will
     // mark the row as needing layout if there was a column logical width
@@ skipping 156 matching lines @@
   // FIXME: Changing the height without a layout can change the overflow so it
   // seems wrong.
 
   unsigned totalRows = m_grid.size();
 
   // Set the width of our section now.  The rows will also be this width.
   setLogicalWidth(table()->contentLogicalWidth());
 
   int vspacing = table()->vBorderSpacing();
   unsigned nEffCols = table()->numEffectiveColumns();
-  LayoutState state(*this, locationOffset());
+  LayoutState state(*this);
 
   // Set the rows' location and size.
   for (unsigned r = 0; r < totalRows; r++) {
     LayoutTableRow* rowLayoutObject = m_grid[r].rowLayoutObject;
     if (rowLayoutObject) {
       rowLayoutObject->setLogicalLocation(LayoutPoint(0, m_rowPos[r]));
       rowLayoutObject->setLogicalWidth(logicalWidth());
       LayoutUnit rowLogicalHeight(m_rowPos[r + 1] - m_rowPos[r] - vspacing);
       if (state.isPaginated() && r + 1 < totalRows) {
         // If the next row has a pagination strut, we need to subtract it. It
@@ skipping 928 matching lines @@
   // the header in all columns.
   // Note that this is in flow thread coordinates, not visual coordinates. The
   // enclosing LayoutFlowThread will convert to visual coordinates.
   if (table()->header() == this && isRepeatingHeaderGroup())
     rect.setHeight(table()->logicalHeight());
   return LayoutTableBoxComponent::mapToVisualRectInAncestorSpace(ancestor, rect,
                                                                  flags);
 }
 
 }  // namespace blink