Bug 1789906 [wpt PR 35837] - blink: Remove 'overflow:clip' UA CSS for root svg elements., a=testonly

khushalsagar · moz-wptsync-bot · commit f5961c7ecf9a · 2022-09-23T22:53:23.000Z
Automatic update from web-platform-tests blink: Remove 'overflow:clip' UA CSS for root svg elements. SVG elements use 'overflow:hidden' via UA CSS to clip to the content-box by default. This should be 'overflow:clip' but hidden ensures backwards compat with flex layout. The used value for painting is still clip. See w3c/csswg-drafts#7714 for details. R=ikilpatrick@chromium.org Bug: 1358820 Change-Id: I79fb2e334ca72807ca6e3b12d85c1f8a33a9f086 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3884796 Auto-Submit: Khushal Sagar <khushalsagar@chromium.org> Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Khushal Sagar <khushalsagar@chromium.org> Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/main@{#1048721} -- wpt-commits: 1647654b1e2904e6f729409e3cea8ed4d6bdde8c wpt-pr: 35837
diff --git a/testing/web-platform/tests/css/css-overflow/display-flex-svg-overflow-default-ref.html b/testing/web-platform/tests/css/css-overflow/display-flex-svg-overflow-default-ref.html
@@ -0,0 +1,16 @@
+<!doctype html>
+<meta charset="utf-8">
+<title>display:flex on svg uses 0 as min size with default overflow</title>
+<link rel="help" href="https://www.w3.org/TR/css-overflow-3/#valdef-overflow">
+<link rel="author" title="Khushal Sagar" href="mailto:khushalsagar@chromium.org">
+<style>
+  body {
+    background: lightblue;
+  }
+  div {
+    width: 100px;
+    height: 100px;
+    background: green;
+  }
+</style>
+<div></div>
diff --git a/testing/web-platform/tests/css/css-overflow/display-flex-svg-overflow-default.html b/testing/web-platform/tests/css/css-overflow/display-flex-svg-overflow-default.html
@@ -0,0 +1,24 @@
+<!doctype html>
+<meta charset="utf-8">
+<title>display:flex on svg uses 0 as min size with default overflow</title>
+<link rel="help" href="https://www.w3.org/TR/css-overflow-3/#valdef-overflow">
+<link rel="author" title="Khushal Sagar" href="mailto:khushalsagar@chromium.org">
+<link rel="match" href="display-flex-svg-overflow-default-ref.html">
+<style>
+  body {
+    background: lightblue;
+  }
+  div {
+    display: flex;
+    width: 100px;
+    height: 100px;
+  }
+  svg {
+    flex-grow: 1;
+  }
+</style>
+<div>
+  <svg>
+    <rect width="150" height="150" fill="green"></rect>
+  </svg>
+</div>
diff --git a/testing/web-platform/tests/css/css-overflow/overflow-clip-x-visible-y-svg.html b/testing/web-platform/tests/css/css-overflow/overflow-clip-x-visible-y-svg.html
@@ -9,6 +9,7 @@
     width: 100px;
     height: 100px;
     overflow-y: visible;
+    overflow-x: clip;
   }
 </style>
 <svg>
diff --git a/testing/web-platform/tests/css/css-overflow/overflow-clip-y-visible-x-svg.html b/testing/web-platform/tests/css/css-overflow/overflow-clip-y-visible-x-svg.html
@@ -9,6 +9,7 @@
     width: 100px;
     height: 100px;
     overflow-x: visible;
+    overflow-y: clip;
   }
 </style>
 <svg>

Original file line number	Diff line number	Diff line change
`@@ -9,6 +9,7 @@`
`9`	`9`	`width: 100px;`
`10`	`10`	`height: 100px;`
`11`	`11`	`overflow-y: visible;`
	`12`	`+ overflow-x: clip;`
`12`	`13`	`}`
`13`	`14`	`</style>`
`14`	`15`	`<svg>`