NG/ContentSize: Save unconstrained intrinsic block size in NG for overflow calc.

vmpstr · chromium-wpt-export-bot · commit 50fa6ade9bda · 2019-10-15T17:22:31.000-04:00
This patch saves the unconstrained intrinsic box size during the block layout algorithm in order to use it for overflow calculation in the node. Since we can override the intrinsic size with css-content-size, we may cause scrollbars to appear. By using unconstrained size, we retain correct behavior in which scrollbars only depend on the child content, and not on content-size override. R=chrishtr@chromium.org, ikilpatrick@chromium.org Bug: 991096 Change-Id: I0d9cb270ee710b759f93202497f69bf72544f81a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1833074 Reviewed-by: Morten Stenshorne <mstensho@chromium.org> Commit-Queue: vmpstr <vmpstr@chromium.org> Cr-Commit-Position: refs/heads/master@{#706135}
diff --git a/css/css-contain/contain-size-scrollbars-004.html b/css/css-contain/contain-size-scrollbars-004.html
@@ -0,0 +1,29 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>CSS Containment Test: Size containment scrollbars</title>
+<link rel="author" title="Vladimir Levin" href="mailto:vmpstr@chromium.org">
+<link rel="help" href="https://drafts.csswg.org/css-contain-1/#containment-size">
+<link rel="match" href="reference/contain-size-scrollbars-004-ref.html">
+<meta name=assert content="This test checks that contain:size element's content and padding are considered for scrollbars.">
+<style>
+#scroller {
+  contain: size;
+  background: lightgreen;
+  overflow: scroll;
+  padding-bottom: 50px;
+  width: 100px;
+  height: 100px;
+}
+#content {
+  background: lightblue;
+  width: 50px;
+  height: 130px;
+}
+</style>
+
+<div id=scroller>
+  <div id=content>
+  </div>
+</div>
+
+<p>This test passes if it has the same output as the reference.</p>
diff --git a/css/css-contain/reference/contain-size-scrollbars-004-ref.html b/css/css-contain/reference/contain-size-scrollbars-004-ref.html
@@ -0,0 +1,25 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>CSS Containment Test: Reference file</title>
+<link rel="author" title="Vladimir Levin" href="mailto:vmpstr@chromium.org">
+<style>
+#scroller {
+  background: lightgreen;
+  overflow: scroll;
+  padding-bottom: 50px;
+  width: 100px;
+  height: 100px;
+}
+#content {
+  background: lightblue;
+  width: 50px;
+  height: 130px;
+}
+</style>
+
+<div id=scroller>
+  <div id=content>
+  </div>
+</div>
+
+<p>This test passes if it has the same output as the reference.</p>
