Cope with intrinsic height styled on table cells

Source/core/layout/LayoutTableCell.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, 2007, 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 82 matching lines @@
     Length styleOrColLogicalWidth() const
     {
         Length styleWidth = style()->logicalWidth();
         if (!styleWidth.isAuto())
             return styleWidth;
         if (LayoutTableCol* firstColumn = table()->colElement(col()))
             return logicalWidthFromColumns(firstColumn, styleWidth);
         return styleWidth;
     }
 
+    int computeLogicalHeightFromCellStyle(const Length& height) const

[mstensho (USE GERRIT), 2015/07/09 08:22:16]

I don't like the name. It essentially means the same thing as the name of the
caller (logicalHeightFromStyle()). Then again, I don't really see the need for a
separate method here, so why not stuff it back into logicalHeightFomStyle()?

+    {
+        int maximumHeight = 0;

[mstensho (USE GERRIT), 2015/07/09 08:22:16]

This one is always 0, so I don't see any need for it.

+        if (height.isIntrinsic())
+            return maximumHeight;
+        return valueForLength(height, maximumHeight);
+    }
+
     int logicalHeightFromStyle() const
     {
-        int styleLogicalHeight = valueForLength(style()->logicalHeight(), 0);

[mstensho (USE GERRIT), 2015/07/09 08:22:16]

Note that "LayoutUnit()" is cheaper than "0", when assigning it to a LayoutUnit.
Something to keep in mind if you make the changes I requested above. :)

+        int styleLogicalHeight = computeLogicalHeightFromCellStyle(style()->logicalHeight());
         // In strict mode, box-sizing: content-box do the right thing and actually add in the border and padding.
         // Call computedCSSPadding* directly to avoid including implicitPadding.
         if (!document().inQuirksMode() && style()->boxSizing() != BORDER_BOX)
             styleLogicalHeight += (computedCSSPaddingBefore() + computedCSSPaddingAfter()).floor() + borderBefore() + borderAfter();
         return styleLogicalHeight;
     }
 
     int logicalHeightForRowSizing() const
     {
         // FIXME: This function does too much work, and is very hot during table layout!
@@ skipping 193 matching lines @@
 
 inline LayoutTableCell* LayoutTableRow::lastCell() const
 {
     ASSERT(children() == virtualChildren());
     return toLayoutTableCell(children()->lastChild());
 }
 
 } // namespace blink
 
 #endif // LayoutTableCell_h