[css-flexbox] Do layout for block axis intrinsic sizes

cbiesinger · chromium-wpt-export-bot · commit 87fea094d82b · 2019-06-03T18:06:22.000-04:00
In the block axis, we do have to do layout for {min,max,fit}-content so we can determine the intrinsic size. MainAxisLengthIsDefinite here is called from ConstructAndAppendFlexItem via ChildHasIntrinsicMainAxisSize (which needs a better name). Bug: 968940 Change-Id: If74e6098280077fefe607e2c5b1f70f133ca1032 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1641510 Reviewed-by: Emil A Eklund <eae@chromium.org> Reviewed-by: David Grogan <dgrogan@chromium.org> Commit-Queue: Christian Biesinger <cbiesinger@chromium.org> Auto-Submit: Christian Biesinger <cbiesinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#665663}
diff --git a/css/css-flexbox/flex-height-min-content.html b/css/css-flexbox/flex-height-min-content.html
@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<title>CSS Flexbox Test: height: min-content is sized correctly</title>
+<link rel="author" title="Google LLC" href="http://www.google.com">
+<link rel="help" href="https://drafts.csswg.org/css-flexbox/#algo-main-item" />
+<link rel="match" href="../reference/ref-filled-green-100px-square.xht" />
+<style>
+#flex {
+  display: flex;
+  flex-direction: column;
+  width: 100px;
+}
+
+#item {
+  min-height: 30px;
+  height: min-content;
+  background: green;
+}
+</style>
+
+<p>Test passes if there is a filled green square and <strong>no red</strong>.</p>
+
+<div id="flex">
+  <div id="item">
+    <div style="height: 100px; width: 100px;"></div>
+  </div>
+</div>
