From 78accb587bab64d45516f58814c64bec99c40368 Mon Sep 17 00:00:00 2001
From: Rune Lillesveen <futhark@chromium.org>
Date: Fri, 6 Jul 2018 21:22:29 +0000
Subject: [PATCH] Bug 1470028 [wpt PR 11594] - Don't clear non-layout-object
 style for PseudoElements., a=testonly

Automatic update from web-platform-testsDon't clear non-layout-object style for PseudoElements.

They are only used for storing an extra ComputedStyle for
display:contents pseudo elements as the ComputedStyle for rendering uses
a display:inline with only properties inherited for the parent element.

Bug: 854563
Change-Id: I4dd3d99b0dee0b538645d005b36347b469799244
Reviewed-on: https://chromium-review.googlesource.com/1107802
Reviewed-by: Anders Ruud <andruud@chromium.org>
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569204}

--

wpt-commits: bcae036590041ef1704e2ba337e0577dac8c11c4
wpt-pr: 11594
---
 testing/web-platform/meta/MANIFEST.json       |  2 +-
 .../css/cssom/getComputedStyle-pseudo.html    | 27 +++++++++++++++++++
 2 files changed, 28 insertions(+), 1 deletion(-)

diff --git a/testing/web-platform/meta/MANIFEST.json b/testing/web-platform/meta/MANIFEST.json
index 7a443e32dc49..3231eebaa262 100644
--- a/testing/web-platform/meta/MANIFEST.json
+++ b/testing/web-platform/meta/MANIFEST.json
@@ -551669,7 +551669,7 @@
    "testharness"
   ],
   "css/cssom/getComputedStyle-pseudo.html": [
-   "d3ef09fb6092078562f8923879b9ece97938df47",
+   "1e5befe5a5494fef67f7c001301f0caddf1aa060",
    "testharness"
   ],
   "css/cssom/historical.html": [
diff --git a/testing/web-platform/tests/css/cssom/getComputedStyle-pseudo.html b/testing/web-platform/tests/css/cssom/getComputedStyle-pseudo.html
index f5a637a7d314..340f9eed9b95 100644
--- a/testing/web-platform/tests/css/cssom/getComputedStyle-pseudo.html
+++ b/testing/web-platform/tests/css/cssom/getComputedStyle-pseudo.html
@@ -44,6 +44,16 @@
   content: "foo";
   position: absolute;
 }
+#contents-pseudos-dynamic::before,
+#contents-pseudos-dynamic::after {
+  display: block;
+  content: "foo";
+  position: absolute;
+}
+#contents-pseudos-dynamic.contents::before,
+#contents-pseudos-dynamic.contents::after {
+  display: contents;
+}
 </style>
 <div id="test">
   <div id="contents"></div>
@@ -51,6 +61,7 @@
   <div id="flex"></div>
   <div id="flex-no-pseudo"></div>
   <div id="contents-pseudos"></div>
+  <div id="contents-pseudos-dynamic"></div>
 </div>
 <script>
 test(function() {
@@ -109,6 +120,22 @@ test(function() {
                   "display: contents in " + pseudo + " should reflect other non-inherited properties in CSSOM");
   });
 }, "display: contents on pseudo-elements");
+test(function() {
+  var contentsPseudosDynamic = document.getElementById('contents-pseudos-dynamic');
+  [":before", ":after"].forEach(function(pseudo) {
+    assert_equals(getComputedStyle(contentsPseudosDynamic, pseudo).display, "block",
+                  "Check that display for " + pseudo + " is block before change");
+  });
+  contentsPseudosDynamic.className = "contents";
+  [":before", ":after"].forEach(function(pseudo) {
+    assert_equals(getComputedStyle(contentsPseudosDynamic, pseudo).display, "contents",
+                  "display: contents in " + pseudo + " should get reflected on CSSOM");
+    assert_equals(getComputedStyle(contentsPseudosDynamic, pseudo).width, "auto",
+                  pseudo + " with display: contents should have no box");
+    assert_equals(getComputedStyle(contentsPseudosDynamic, pseudo).position, "absolute",
+                  "display: contents in " + pseudo + " should reflect other non-inherited properties in CSSOM");
+  });
+}, "Dynamically change to display: contents on pseudo-elements");
 test(function() {
   var div = document.getElementById('test');
   assert_throws(new TypeError(), () => getComputedStyle(div, "totallynotapseudo"),