[css-flexbox] logical widths are always definite.

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

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 743 matching lines @@
     return isHorizontalFlow() ? child.borderAndPaddingWidth() : child.borderAndPaddingHeight();
 }
 
 bool LayoutFlexibleBox::mainAxisLengthIsDefinite(const LayoutBox& child, const Length& flexBasis) const
 {
     if (flexBasis.isAuto())
         return false;
     if (flexBasis.hasPercent()) {
         return isColumnFlow() ?
             child.computePercentageLogicalHeight(flexBasis) != -1 :
-            hasDefiniteLogicalWidth();
+            true;
     }
     return true;
 }
 
 bool LayoutFlexibleBox::crossAxisLengthIsDefinite(const LayoutBox& child, const Length& length) const
 {
     if (length.isAuto())
         return false;
     if (length.hasPercent()) {
         return hasOrthogonalFlow(child) ?
-            hasDefiniteLogicalWidth() :
+            true :
             child.computePercentageLogicalHeight(length) != -1;
     }
     // TODO(cbiesinger): Eventually we should support other types of sizes here. Requires updating
     // computeMainSizeFromAspectRatioUsing.
     return length.isFixed();
 }
 
 bool LayoutFlexibleBox::childFlexBaseSizeRequiresLayout(const LayoutBox& child) const
 {
     return !mainAxisLengthIsDefinite(child, flexBasisForChild(child)) && (
@@ skipping 314 matching lines @@
             transferredSize = adjustChildSizeForAspectRatioCrossAxisMinAndMax(child, transferredSize);
             minExtent = std::min(transferredSize, contentSize);
         } else {
             minExtent = contentSize;
         }
     }
     ASSERT(minExtent >= 0);
     return std::max(childSize, minExtent);
 }
 
-LayoutUnit LayoutFlexibleBox::computeDefiniteLogicalWidth()
-{
-    const Length& widthLength = styleRef().logicalWidth();
-    if (widthLength.hasPercent() && !hasDefiniteLogicalWidth())
-        return LayoutUnit(-1);
-
-    if (widthLength.isAuto()) {
-        // We can still have a definite width even with width: auto if we're a flex item ourselves
-        if (!isFlexItem())
-            return LayoutUnit(-1);
-        return toLayoutFlexibleBox(parent())->childLogicalWidthForPercentageResolution(*this);
-    }
-    LogicalExtentComputedValues computedValues;
-    computeLogicalWidth(computedValues);
-    return computedValues.m_extent;
-}
-
-LayoutUnit LayoutFlexibleBox::computeDefiniteLogicalHeight()
-{
-    const Length& heightLength = styleRef().logicalHeight();
-    if (heightLength.hasPercent()) {
-        return computePercentageLogicalHeight(heightLength);
-    }
-    if (heightLength.isAuto()) {
-        // We can still have a definite height even with height: auto if we're a flex item ourselves
-        if (!isFlexItem())
-            return LayoutUnit(-1);
-        return toLayoutFlexibleBox(parent())->childLogicalHeightForPercentageResolution(*this);
-    }
-    LogicalExtentComputedValues computedValues;
-    computeLogicalHeight(LayoutUnit(-1), LayoutUnit(), computedValues);
-    return computedValues.m_extent;
-}
-
 LayoutUnit LayoutFlexibleBox::crossSizeForPercentageResolution(const LayoutBox& child)
 {
     if (alignmentForChild(child) != ItemPositionStretch)
         return LayoutUnit(-1);
 
     // Here we implement https://drafts.csswg.org/css-flexbox/#algo-stretch
     if (hasOrthogonalFlow(child) && child.hasOverrideLogicalContentWidth())
         return child.overrideLogicalContentWidth();
     if (!hasOrthogonalFlow(child) && child.hasOverrideLogicalContentHeight())
         return child.overrideLogicalContentHeight();
@@ skipping 28 matching lines @@
     return child.hasOverrideLogicalContentWidth() ? child.overrideLogicalContentWidth() : LayoutUnit(-1);
 }
 
 LayoutUnit LayoutFlexibleBox::childLogicalHeightForPercentageResolution(const LayoutBox& child)
 {
     if (!hasOrthogonalFlow(child))
         return crossSizeForPercentageResolution(child);
     return mainSizeForPercentageResolution(child);
 }
 
-LayoutUnit LayoutFlexibleBox::childLogicalWidthForPercentageResolution(const LayoutBox& child)
-{
-    if (hasOrthogonalFlow(child))
-        return crossSizeForPercentageResolution(child);
-    return mainSizeForPercentageResolution(child);
-}
-
 LayoutUnit LayoutFlexibleBox::adjustChildSizeForAspectRatioCrossAxisMinAndMax(const LayoutBox& child, LayoutUnit childSize)
 {
     Length crossMin = isHorizontalFlow() ? child.style()->minHeight() : child.style()->minWidth();
     Length crossMax = isHorizontalFlow() ? child.style()->maxHeight() : child.style()->maxWidth();
 
 
     if (crossAxisLengthIsDefinite(child, crossMax)) {
         LayoutUnit maxValue = computeMainSizeFromAspectRatioUsing(child, crossMax);
         childSize = std::min(maxValue, childSize);
     }
@@ skipping 694 matching lines @@
             ASSERT(child);
             LayoutUnit lineCrossAxisExtent = lineContexts[lineNumber].crossAxisExtent;
             LayoutUnit originalOffset = lineContexts[lineNumber].crossAxisOffset - crossAxisStartEdge;
             LayoutUnit newOffset = contentExtent - originalOffset - lineCrossAxisExtent;
             adjustAlignmentForChild(*child, newOffset - originalOffset);
         }
     }
 }
 
 } // namespace blink