[LayoutNG] Set ShrinkToFit when determining MinMaxSize via layout

This is a reland of 4fb5e9ec which should be fine after 4548e4da landed. Bug: 635619 Change-Id: I374a1f4bc4d8c0ea4fbb91cf9f560ff145c371db Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2024467 Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Reviewed-by: Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#735803}

[LayoutNG] Set ShrinkToFit when determining MinMaxSize via layout
This is a reland of 4fb5e9ec which should be fine after 4548e4da landed. Bug: 635619 Change-Id: I374a1f4bc4d8c0ea4fbb91cf9f560ff145c371db Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2024467 Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Reviewed-by: Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#735803}
134b2acd · David Grogan · Commit Bot · 43a32353 · 134b2acd · 134b2acd
Commit 134b2acd authored Jan 28, 2020 by David Grogan Committed by Commit Bot Jan 28, 2020
3 changed files
--- a/third_party/blink/renderer/core/layout/ng/ng_space_utils.cc
+++ b/third_party/blink/renderer/core/layout/ng/ng_space_utils.cc
@@ -38,6 +38,7 @@ NGConstraintSpace CreateIndefiniteConstraintSpaceForChild(
  builder.SetAvailableSize(indefinite_size);
  builder.SetPercentageResolutionSize(indefinite_size);
  builder.SetReplacedPercentageResolutionSize(indefinite_size);
+  builder.SetIsShrinkToFit(child.Style().LogicalWidth().IsAuto());
  return builder.ToConstraintSpace();
 }


--- a/third_party/blink/web_tests/FlagExpectations/disable-layout-ng
+++ b/third_party/blink/web_tests/FlagExpectations/disable-layout-ng
@@ -95,6 +95,7 @@ crbug.com/591099 external/wpt/css/css-sizing/clone-nowrap-intrinsic-size-bidi.ht
 crbug.com/591099 external/wpt/css/css-sizing/intrinsic-percent-non-replaced-004.html [ Failure ]
 crbug.com/591099 external/wpt/css/css-sizing/intrinsic-percent-non-replaced-005.html [ Failure ]
 crbug.com/591099 external/wpt/css/css-sizing/whitespace-and-break.html [ Failure ]
+crbug.com/591099 external/wpt/css/css-sizing/ortho-writing-mode-001.html [ Failure ]

 ### external/wpt/css/css-text/boundary-shaping/
 crbug.com/591099 external/wpt/css/css-text/boundary-shaping/boundary-shaping-001.html [ Failure ]

--- a/third_party/blink/web_tests/external/wpt/css/css-sizing/ortho-writing-mode-001.html
+++ b/third_party/blink/web_tests/external/wpt/css/css-sizing/ortho-writing-mode-001.html
+<!DOCTYPE html>
+<link rel="author" title="David Grogan" href="mailto:dgrogan@chromium.org">
+<link rel="author" title="Morten Stenshorne" href="mailto:mstensho@chromium.org">
+<link rel="help" href="https://drafts.csswg.org/css-sizing-3/#intrinsic-sizes" title="first paragraph">
+<link rel="help" href="https://www.w3.org/TR/css-writing-modes-4/#orthogonal-auto">
+<link rel="match" href="../reference/ref-filled-green-100px-square.xht">
+<meta name="flags" content="" />
+<meta name="assert" content="The intrinsic size of an orthogonal child is its shrink to fit size after layout" />
+
+<p>Test passes if there is a filled green square and <strong>no red</strong>.</p>
+
+<div style="width:min-content; background:red;">
+  <div style="writing-mode:vertical-rl; background:green;">
+    <div style="margin-right:50%; width:50px; height:100px;"></div>
+  </div>
+</div>