Bug 1541742 [wpt PR 16193] - [css-contain] Size containment should include scrollbars, a=testonly
authorManuel Rego Casasnovas <rego@igalia.com>
Fri, 17 May 2019 11:02:24 +0000
changeset 477086 028ccbf8c353021f718acf933c39b8966ba310a2
parent 477085 19e26c39ac5beee2bd658e5bbf12ca9cb4150cba
child 477087 a62f7463b2b8ec8a5380552282ab13f47aca61ca
push id36116
push usershindli@mozilla.com
push dateThu, 06 Jun 2019 10:00:05 +0000
treeherdermozilla-central@fee989d27558 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstestonly
bugs1541742, 16193, 948300, 1547797, 646532
milestone69.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1541742 [wpt PR 16193] - [css-contain] Size containment should include scrollbars, a=testonly Automatic update from web-platform-tests [css-contain] Size containment should include scrollbars Like it happens on regular elements, flexbox and grid containers with "contain: size" should include scrollbars width when computing the intrinsic sizes. BUG=948300 TEST=external/wpt/css/css-contain/contain-size-scrollbars-001.html TEST=external/wpt/css/css-contain/contain-size-scrollbars-002.html TEST=external/wpt/css/css-contain/contain-size-scrollbars-003.html Change-Id: If044632469e3c2500aac155d04f9a319930ef6e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547797 Reviewed-by: Christian Biesinger <cbiesinger@chromium.org> Reviewed-by: Javier Fernandez <jfernandez@igalia.com> Commit-Queue: Manuel Rego <rego@igalia.com> Cr-Commit-Position: refs/heads/master@{#646532} -- wpt-commits: e636026448850ef30983a79f79f74b9bb8a1cd53 wpt-pr: 16193
testing/web-platform/tests/css/css-contain/contain-size-scrollbars-001.html
testing/web-platform/tests/css/css-contain/contain-size-scrollbars-002.html
testing/web-platform/tests/css/css-contain/contain-size-scrollbars-003.html
--- a/testing/web-platform/tests/css/css-contain/contain-size-scrollbars-001.html
+++ b/testing/web-platform/tests/css/css-contain/contain-size-scrollbars-001.html
@@ -9,9 +9,9 @@
 div {
   contain: size;
   display: inline-block;
   border: solid thick;
   overflow: scroll;
 }
 </style>
 <p>This test passes if it has the same output as the reference.</p>
-<div></div>
+<div>FAIL</div>
new file mode 100644
--- /dev/null
+++ b/testing/web-platform/tests/css/css-contain/contain-size-scrollbars-002.html
@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>CSS Containment Test: Size containment scrollbars</title>
+<link rel="author" title="Manuel Rego Casasnovas" href="mailto:rego@igalia.com">
+<link rel="help" href="https://drafts.csswg.org/css-contain-1/#containment-size">
+<link rel="match" href="reference/contain-size-scrollbars-001-ref.html">
+<meta name=assert content="This test checks that the size of a flexbox container with 'contain: size' includes the scrollbars too.">
+<style>
+div {
+  contain: size;
+  display: inline-flex;
+  border: solid thick;
+  overflow: scroll;
+}
+</style>
+<p>This test passes if it has the same output as the reference.</p>
+<div>FAIL</div>
new file mode 100644
--- /dev/null
+++ b/testing/web-platform/tests/css/css-contain/contain-size-scrollbars-003.html
@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>CSS Containment Test: Size containment scrollbars</title>
+<link rel="author" title="Manuel Rego Casasnovas" href="mailto:rego@igalia.com">
+<link rel="help" href="https://drafts.csswg.org/css-contain-1/#containment-size">
+<link rel="match" href="reference/contain-size-scrollbars-001-ref.html">
+<meta name=assert content="This test checks that the size of a grid container with 'contain: size' includes the scrollbars too.">
+<style>
+div {
+  contain: size;
+  display: inline-grid;
+  border: solid thick;
+  overflow: scroll;
+}
+</style>
+<p>This test passes if it has the same output as the reference.</p>
+<div>FAIL</div>