autoland: changeset 531141:8db0cdde1d281d407b6df3f94b715417e2d76480

author	Manuel Rego Casasnovas <rego@igalia.com>
	Wed, 13 May 2020 09:45:43 +0000
changeset 531141	8db0cdde1d281d407b6df3f94b715417e2d76480
parent 531140	bbff8a3ac01236bd57bde28f8f842e81e8e2cc48
child 531142	56b03c5d981273f70e61294027c763cbac6a351c
push id	116503
push user	wptsync@mozilla.com
push date	Wed, 20 May 2020 10:37:27 +0000
treeherder	autoland@135acd548abf [default view] [failures only]
perfherder	[talos] [build metrics] [platform microbench] (compared to previous push)
reviewers	testonly
bugs	1636045, 23420, 1078539, 2182471, 766312
milestone	78.0a1
first release with	nightly linux32 6c10970490f3 / 78.0a1 / 20200520213050 / files nightly linux64 6c10970490f3 / 78.0a1 / 20200520213050 / files nightly mac 6c10970490f3 / 78.0a1 / 20200520213050 / files nightly win32 6c10970490f3 / 78.0a1 / 20200520213050 / files nightly win64 6c10970490f3 / 78.0a1 / 20200520213050 / files
last release without	nightly linux32 855249e545c3 / 78.0a1 / 20200520033931 / files nightly linux64 855249e545c3 / 78.0a1 / 20200520033931 / files nightly mac 855249e545c3 / 78.0a1 / 20200520033931 / files nightly win32 855249e545c3 / 78.0a1 / 20200520033931 / files nightly win64 855249e545c3 / 78.0a1 / 20200520033931 / files

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

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>
+

testing/web-platform/tests/css/css-ui/outline-024.html		file \| annotate \| diff \| comparison \| revisions
testing/web-platform/tests/css/css-ui/outline-025.html		file \| annotate \| diff \| comparison \| revisions
testing/web-platform/tests/css/css-ui/outline-026.html		file \| annotate \| diff \| comparison \| revisions
testing/web-platform/tests/css/css-ui/reference/outline-024-ref.html		file \| annotate \| diff \| comparison \| revisions