Treat cells with percent height as auto if their container depends on content height

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 1096 matching lines @@
 
   int remaining_extra_logical_height = extra_logical_height;
   DistributeExtraLogicalHeightToPercentRows(remaining_extra_logical_height,
                                             total_percent);
   DistributeExtraLogicalHeightToAutoRows(remaining_extra_logical_height,
                                          auto_rows_count);
   DistributeRemainingExtraLogicalHeight(remaining_extra_logical_height);
   return extra_logical_height - remaining_extra_logical_height;
 }
 
+bool CellHasExplicitlySpecifiedHeight(const LayoutTableCell& cell) {
+  if (cell.Style()->LogicalHeight().IsFixed())
+    return true;
+  LayoutBlock* cb = cell.ContainingBlock();
+  if (cb->AvailableLogicalHeightForPercentageComputation() == -1)
+    return false;
+  return true;
+}
+
 static bool ShouldFlexCellChild(const LayoutTableCell& cell,
                                 LayoutObject* cell_descendant) {
-  if (!cell.Style()->LogicalHeight().IsSpecified())
+  if (!CellHasExplicitlySpecifiedHeight(cell))
     return false;
   if (cell_descendant->Style()->OverflowY() == EOverflow::kVisible ||
       cell_descendant->Style()->OverflowY() == EOverflow::kHidden)
     return true;
   return cell_descendant->IsBox() &&
          ToLayoutBox(cell_descendant)->ShouldBeConsideredAsReplaced();
 }
 
 void LayoutTableSection::LayoutRows() {
 #if DCHECK_IS_ON()
@@ skipping 824 matching lines @@
   // do, but it will clip the cells that spill out of the table section.
   // strict mode, Mozilla and WinIE both regrow the table to accommodate the
   // new height of the cell (thus letting the percentages cause growth one
   // time only). We may also not be handling row-spanning cells correctly.
   //
   // Note also the oddity where replaced elements always flex, and yet blocks/
   // tables do not necessarily flex. WinIE is crazy and inconsistent, and we
   // can't hope to match the behavior perfectly, but we'll continue to refine it
   // as we discover new bugs. :)
   bool cell_children_flex = false;
-  bool flex_all_children = cell.Style()->LogicalHeight().IsSpecified() ||
+  bool flex_all_children = CellHasExplicitlySpecifiedHeight(cell) ||
                            (!Table()->Style()->LogicalHeight().IsAuto() &&
                             row_height != cell.LogicalHeight());
 
   for (LayoutObject* child = cell.FirstChild(); child;
        child = child->NextSibling()) {
     if (!child->IsText() && child->Style()->LogicalHeight().IsPercentOrCalc() &&
         (flex_all_children || ShouldFlexCellChild(cell, child)) &&
         (!child->IsTable() || ToLayoutTable(child)->HasSections())) {
       cell_children_flex = true;
       break;
@@ skipping 183 matching lines @@
 bool LayoutTableSection::PaintedOutputOfObjectHasNoEffectRegardlessOfSize()
     const {
   // LayoutTableSection paints background from columns.
   if (Table()->HasColElements())
     return false;
   return LayoutTableBoxComponent::
       PaintedOutputOfObjectHasNoEffectRegardlessOfSize();
 }
 
 }  // namespace blink