Constrain width/height to minmax values

third_party/WebKit/Source/core/layout/ng/ng_absolute_utils.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/layout/ng/ng_absolute_utils.h"
 
 #include "core/layout/ng/ng_constraint_space.h"
 #include "core/layout/ng/ng_length_utils.h"
 #include "core/style/ComputedStyle.h"
 #include "platform/LengthFunctions.h"
 
 namespace blink {
 
 namespace {
 
 bool AbsoluteHorizontalNeedsEstimate(const ComputedStyle& style) {
   Length width = style.width();
   return width.isIntrinsic() ||
          (width.isAuto() && (style.left().isAuto() || style.right().isAuto()));
 }
 
 bool AbsoluteVerticalNeedsEstimate(const ComputedStyle& style) {
   Length height = style.height();
   return height.isIntrinsic() ||
          (height.isAuto() && (style.top().isAuto() || style.bottom().isAuto()));
 }
 
+LayoutUnit ResolveWidth(const Length& width,
+                        const NGConstraintSpace& space,
+                        const ComputedStyle& style,
+                        const Optional<MinAndMaxContentSizes>& child_minmax) {
+  if (space.WritingMode() == kHorizontalTopBottom)
+    return ResolveInlineLength(space, style, child_minmax, width,
+                               LengthResolveType::kContentSize);

[cbiesinger, 2017/01/19 22:49:13]

So, you really need to pass in the LengthResolveType so that you can handle
kContentSize vs kMinSize vs kMaxSize correctly.

[atotic, 2017/01/19 23:47:43]

done. The usage of Resolve.. methods keeps tripping me.

+  LayoutUnit computed_width =
+      child_minmax.has_value() ? child_minmax->max_content : LayoutUnit();
+  return ResolveBlockLength(space, style, style.width(), computed_width,
+                            LengthResolveType::kContentSize);
+}
+
 // Implement absolute horizontal size resolution algorithm.
 // https://www.w3.org/TR/css-position-3/#abs-non-replaced-width
 void ComputeAbsoluteHorizontal(
     const NGConstraintSpace& space,
     const ComputedStyle& style,
     const NGStaticPosition& static_position,
     const Optional<MinAndMaxContentSizes>& child_minmax,
     NGAbsolutePhysicalPosition* position) {
   NGLogicalSize percentage_logical = space.PercentageResolutionSize();
   NGPhysicalSize percentage_physical =
@@ skipping 13 matching lines @@
     margin_right =
         valueForLength(style.marginRight(), percentage_logical.inline_size);
   Optional<LayoutUnit> left;
   if (!style.left().isAuto())
     left = valueForLength(style.left(), percentage_physical.width);
   Optional<LayoutUnit> right;
   if (!style.right().isAuto())
     right = valueForLength(style.right(), percentage_physical.width);
   LayoutUnit border_padding =
       border_left + border_right + padding_left + padding_right;
+  Optional<LayoutUnit> min_width;
+  if (!style.minWidth().isAuto())
+    min_width = ResolveWidth(style.minWidth(), space, style, child_minmax);
+  Optional<LayoutUnit> max_width;
+  if (!style.maxWidth().isMaxSizeNone())
+    max_width = ResolveWidth(style.maxWidth(), space, style, child_minmax);
   Optional<LayoutUnit> width;
   if (!style.width().isAuto()) {
-    if (space.WritingMode() == kHorizontalTopBottom) {
-      width = ResolveInlineLength(space, style, child_minmax, style.width(),
-                                  LengthResolveType::kContentSize);
-    } else {
-      LayoutUnit computed_width =
-          child_minmax.has_value() ? child_minmax->max_content : LayoutUnit();
-      width = ResolveBlockLength(space, style, style.width(), computed_width,
-                                 LengthResolveType::kContentSize);
-    }
+    width = ResolveWidth(style.width(), space, style, child_minmax);
+    width = ConstrainByMinMax(*width, min_width, max_width);
   }
-
   NGPhysicalSize container_size =
       space.AvailableSize().ConvertToPhysical(space.WritingMode());
   DCHECK(container_size.width != NGSizeIndefinite);
 
   // Solving the equation:
   // left + marginLeft + width + marginRight + right  = container width
   if (!left && !right && !width) {
     // Standard: "If all three of left, width, and right are auto:"
     if (!margin_left)
       margin_left = LayoutUnit();
     if (!margin_right)
       margin_right = LayoutUnit();
     DCHECK(child_minmax.has_value());
     width = child_minmax->ShrinkToFit(container_size.width);
+    width = ConstrainByMinMax(*width, min_width, max_width);
     if (space.Direction() == TextDirection::kLtr) {
       left = static_position.LeftPosition(container_size.width, *width,
                                           *margin_left, *margin_right);
     } else {
       right = static_position.RightPosition(container_size.width, *width,
                                             *margin_left, *margin_right);
     }
   } else if (left && right && width) {
     // Standard: "If left, right, and width are not auto:"
     // Compute margins.
@@ skipping 35 matching lines @@
     margin_left = LayoutUnit();
   if (!margin_right)
     margin_right = LayoutUnit();
 
   // Rules 1 through 3, 2 out of 3 are unknown.
   if (!left && !width) {
     // Rule 1: left/width are unknown.
     DCHECK(right.has_value());
     DCHECK(child_minmax.has_value());
     width = child_minmax->ShrinkToFit(container_size.width);
+    width = ConstrainByMinMax(*width, min_width, max_width);
   } else if (!left && !right) {
     // Rule 2.
     DCHECK(width.has_value());
     if (space.Direction() == TextDirection::kLtr)
       left = static_position.LeftPosition(container_size.width, *width,
                                           *margin_left, *margin_right);
     else
       right = static_position.RightPosition(container_size.width, *width,
                                             *margin_left, *margin_right);
   } else if (!width && !right) {
     // Rule 3.
     DCHECK(child_minmax.has_value());
     width = child_minmax->ShrinkToFit(container_size.width);
+    width = ConstrainByMinMax(*width, min_width, max_width);
   }
 
   // Rules 4 through 6, 1 out of 3 are unknown.
   if (!left) {
     left =
         container_size.width - *right - *width - *margin_left - *margin_right;
   } else if (!right) {
     right =
         container_size.width - *left - *width - *margin_left - *margin_right;
   } else if (!width) {
     width =
         container_size.width - *left - *right - *margin_left - *margin_right;
   }
   DCHECK_EQ(container_size.width,
             *left + *right + *margin_left + *margin_right + *width);
 
+  width = ConstrainByMinMax(*width, min_width, max_width);
   // Negative widths are not allowed.
   width = std::max(*width, border_padding);
 
   position->inset.left = *left + *margin_left;
   position->inset.right = *right + *margin_right;
   position->size.width = *width;
 }
 
+LayoutUnit ResolveHeight(const Length& height,

[cbiesinger, 2017/01/19 22:49:13]

I would move this function to the top, under ResolveWidth

[atotic, 2017/01/19 23:47:43]

done.

+                         const NGConstraintSpace& space,
+                         const ComputedStyle& style,
+                         const Optional<MinAndMaxContentSizes>& child_minmax) {
+  if (space.WritingMode() != kHorizontalTopBottom)
+    return ResolveInlineLength(space, style, child_minmax, height,
+                               LengthResolveType::kContentSize);
+  LayoutUnit computed_height =
+      child_minmax.has_value() ? child_minmax->max_content : LayoutUnit();
+  return ResolveBlockLength(space, style, height, computed_height,
+                            LengthResolveType::kContentSize);
+}
 // Implements absolute vertical size resolution algorithm.
 // https://www.w3.org/TR/css-position-3/#abs-non-replaced-height
 void ComputeAbsoluteVertical(
     const NGConstraintSpace& space,
     const ComputedStyle& style,
     const NGStaticPosition& static_position,
     const Optional<MinAndMaxContentSizes>& child_minmax,
     NGAbsolutePhysicalPosition* position) {
   NGLogicalSize percentage_logical = space.PercentageResolutionSize();
   NGPhysicalSize percentage_physical =
@@ skipping 15 matching lines @@
         valueForLength(style.marginBottom(), percentage_logical.inline_size);
   Optional<LayoutUnit> top;
   if (!style.top().isAuto())
     top = valueForLength(style.top(), percentage_physical.height);
   Optional<LayoutUnit> bottom;
   if (!style.bottom().isAuto())
     bottom = valueForLength(style.bottom(), percentage_physical.height);
   LayoutUnit border_padding =
       border_top + border_bottom + padding_top + padding_bottom;
 
+  Optional<LayoutUnit> min_height;
+  if (!style.minHeight().isAuto())
+    min_height = ResolveHeight(style.minHeight(), space, style, child_minmax);
+  Optional<LayoutUnit> max_height;
+  if (!style.maxHeight().isMaxSizeNone())
+    max_height = ResolveHeight(style.maxHeight(), space, style, child_minmax);
   Optional<LayoutUnit> height;
   if (!style.height().isAuto()) {
-    if (space.WritingMode() == kHorizontalTopBottom) {
-      LayoutUnit computed_height =
-          child_minmax.has_value() ? child_minmax->max_content : LayoutUnit();
-      height = ResolveBlockLength(space, style, style.height(), computed_height,
-                                  LengthResolveType::kContentSize);
-    } else {
-      height = ResolveInlineLength(space, style, child_minmax, style.height(),
-                                   LengthResolveType::kContentSize);
-    }
+    height = ResolveHeight(style.height(), space, style, child_minmax);
+    height = ConstrainByMinMax(*height, min_height, max_height);
   }
 
   NGPhysicalSize container_size =
       space.AvailableSize().ConvertToPhysical(space.WritingMode());
   DCHECK(container_size.height != NGSizeIndefinite);
 
   // Solving the equation:
   // top + marginTop + height + marginBottom + bottom
   // + border_padding = container height
   if (!top && !bottom && !height) {
     // Standard: "If all three of top, height, and bottom are auto:"
     if (!margin_top)
       margin_top = LayoutUnit();
     if (!margin_bottom)
       margin_bottom = LayoutUnit();
     DCHECK(child_minmax.has_value());
     height = child_minmax->ShrinkToFit(container_size.height);
+    height = ConstrainByMinMax(*height, min_height, max_height);
     top = static_position.TopPosition(container_size.height, *height,
                                       *margin_top, *margin_bottom);
   } else if (top && bottom && height) {
     // Standard: "If top, bottom, and height are not auto:"
     // Compute margins.
     LayoutUnit margin_space = container_size.height - *top - *bottom - *height;
     // When both margins are auto.
     if (!margin_top && !margin_bottom) {
       if (margin_space > 0) {
         margin_top = margin_space / 2;
@@ skipping 19 matching lines @@
     margin_top = LayoutUnit();
   if (!margin_bottom)
     margin_bottom = LayoutUnit();
 
   // Rules 1 through 3, 2 out of 3 are unknown, fix 1.
   if (!top && !height) {
     // Rule 1.
     DCHECK(bottom.has_value());
     DCHECK(child_minmax.has_value());
     height = child_minmax->ShrinkToFit(container_size.height);
+    height = ConstrainByMinMax(*height, min_height, max_height);
   } else if (!top && !bottom) {
     // Rule 2.
     DCHECK(height.has_value());
     top = static_position.TopPosition(container_size.height, *height,
                                       *margin_top, *margin_bottom);
   } else if (!height && !bottom) {
     // Rule 3.
     DCHECK(child_minmax.has_value());
     height = child_minmax->ShrinkToFit(container_size.height);
+    height = ConstrainByMinMax(*height, min_height, max_height);
   }
 
   // Rules 4 through 6, 1 out of 3 are unknown.
   if (!top) {
     top = container_size.height - *bottom - *height - *margin_top -
           *margin_bottom;
   } else if (!bottom) {
     bottom =
         container_size.height - *top - *height - *margin_top - *margin_bottom;
   } else if (!height) {
     height =
         container_size.height - *top - *bottom - *margin_top - *margin_bottom;
   }
   DCHECK_EQ(container_size.height,
             *top + *bottom + *margin_top + *margin_bottom + *height);
 
+  height = ConstrainByMinMax(*height, min_height, max_height);
   // Negative heights are not allowed.
   height = std::max(*height, border_padding);
 
   position->inset.top = *top + *margin_top;
   position->inset.bottom = *bottom + *margin_bottom;
   position->size.height = *height;
 }
 
 }  // namespace
 
@@ skipping 62 matching lines @@
   if (style.isHorizontalWritingMode())
     ComputeAbsoluteVertical(space, style, static_position, child_minmax,
                             position);
   else {
     ComputeAbsoluteHorizontal(space, style, static_position, child_minmax,
                               position);
   }
 }
 
 }  // namespace blink