Bug 1636045 [wpt PR 23420] - Fix "auto" outline with padding and overflowing inline descendants, a=testonly
authorManuel Rego Casasnovas <rego@igalia.com>
Wed, 13 May 2020 09:45:43 +0000
changeset 531124 8db0cdde1d281d407b6df3f94b715417e2d76480
parent 531123 bbff8a3ac01236bd57bde28f8f842e81e8e2cc48
child 531125 56b03c5d981273f70e61294027c763cbac6a351c
push id37435
push userapavel@mozilla.com
push dateWed, 20 May 2020 15:28:23 +0000
treeherdermozilla-central@5415da14ec9a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstestonly
bugs1636045, 23420, 1078539, 2182471, 766312
milestone78.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 1636045 [wpt PR 23420] - Fix "auto" outline with padding and overflowing inline descendants, a=testonly Automatic update from web-platform-tests Fix "auto" outline with padding and overflowing inline descendants This patch fixes a regression in LayoutNG compared to legacy, when the descendant of an element with padding and "outline: auto" is an inline block. And also when it's an inline element (which was not working properly in legacy layout either, that's why outline-025 and outline-026 tests fail on legacy). We need to rebaseline an old pixel test, as we're coming back to the legacy layout behavior for it. BUG=1078539 TEST=external/wpt/css/css-ui/outline-024.html TEST=external/wpt/css/css-ui/outline-025.html TEST=external/wpt/css/css-ui/outline-026.html Change-Id: I8ccf72fb34e1d9ce6821eec9312f1fecd52e1281 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2182471 Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by: Koji Ishii <kojii@chromium.org> Commit-Queue: Manuel Rego <rego@igalia.com> Cr-Commit-Position: refs/heads/master@{#766312} -- wpt-commits: 663e39d65620ff336e5183b0b4c2a7bb1636c2c7 wpt-pr: 23420
testing/web-platform/tests/css/css-ui/outline-024.html
testing/web-platform/tests/css/css-ui/outline-025.html
testing/web-platform/tests/css/css-ui/outline-026.html
testing/web-platform/tests/css/css-ui/reference/outline-024-ref.html
new file mode 100644
--- /dev/null
+++ b/testing/web-platform/tests/css/css-ui/outline-024.html
@@ -0,0 +1,13 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>CSS Basic User Interface: outline with padding</title>
+<link rel="author" title="Manuel Rego Casasnovas" href="mailto:rego@igalia.com">
+<link rel="help" href="https://drafts.csswg.org/css-ui/#outline-props">
+<link rel="match" href="reference/outline-024-ref.html">
+<meta name="assert" content="Test checks that 'auto' outline works as expected in an element with padding and overflowing inline block descendant.">
+<meta name="flags" content="ahem">
+<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
+<p>Test passes if the outline is wrapping all the green boxes.</p>
+<div style="width: 50px; height: 50px; padding: 10px; outline: auto;">
+  <div style="display: inline-block; font: 50px/1 Ahem; color: green;">XX</div>
+</div>
new file mode 100644
--- /dev/null
+++ b/testing/web-platform/tests/css/css-ui/outline-025.html
@@ -0,0 +1,13 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>CSS Basic User Interface: outline with padding</title>
+<link rel="author" title="Manuel Rego Casasnovas" href="mailto:rego@igalia.com">
+<link rel="help" href="https://drafts.csswg.org/css-ui/#outline-props">
+<link rel="match" href="reference/outline-024-ref.html">
+<meta name="assert" content="Test checks that 'auto' outline works as expected in an element with padding and overflowing inline descendant.">
+<meta name="flags" content="ahem">
+<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
+<p>Test passes if the outline is wrapping all the green boxes.</p>
+<div style="width: 50px; height: 50px; padding: 10px; outline: auto;">
+  <div style="display: inline; font: 50px/1 Ahem; color: green;">XX</div>
+</div>
new file mode 100644
--- /dev/null
+++ b/testing/web-platform/tests/css/css-ui/outline-026.html
@@ -0,0 +1,13 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>CSS Basic User Interface: outline with padding</title>
+<link rel="author" title="Manuel Rego Casasnovas" href="mailto:rego@igalia.com">
+<link rel="help" href="https://drafts.csswg.org/css-ui/#outline-props">
+<link rel="match" href="reference/outline-024-ref.html">
+<meta name="assert" content="Test checks that 'auto' outline works as expected in an element with padding and overflowing SPAN descendant.">
+<meta name="flags" content="ahem">
+<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
+<p>Test passes if the outline is wrapping all the green boxes.</p>
+<div style="width: 50px; height: 50px; padding: 10px; outline: auto;">
+  <span style="font: 50px/1 Ahem; color: green;">XX</span>
+</div>
new file mode 100644
--- /dev/null
+++ b/testing/web-platform/tests/css/css-ui/reference/outline-024-ref.html
@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>CSS Basic User Interface: Reference test for outline with padding</title>
+<link rel="author" title="Manuel Rego Casasnovas" href="mailto:rego@igalia.com">
+<meta name="flags" content="ahem">
+<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
+<p>Test passes if the outline is wrapping all the green boxes.</p>
+<div style="width: 50px; height: 50px; padding: 10px; outline: auto;">
+  <div style="font: 50px/1 Ahem; color: green;">XX</div>
+</div>
+