From f08b13d6b4bf83c6f21abbd03ac2d1f7517a8c81 Mon Sep 17 00:00:00 2001
From: Marco Castelluccio <mcastelluccio@mozilla.com>
Date: Fri, 13 Dec 2024 03:50:41 +0000
Subject: [PATCH] Bug 1935777 [wpt PR 49576] - Remove :closed pseudo-class,
 a=testonly

Automatic update from web-platform-tests
Remove :closed pseudo-class

This was resolved here:
https://github.com/w3c/csswg-drafts/issues/11039#issuecomment-2491673891

Change-Id: Icfe2bbf5a7b87c50a52cedae98021425a5121429
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6049400
Reviewed-by: Tommy Nyquist <nyquistchromium.org>
Auto-Submit: Joey Arhar <jarharchromium.org>
Commit-Queue: Joey Arhar <jarharchromium.org>
Reviewed-by: David Baron <dbaronchromium.org>
Cr-Commit-Position: refs/heads/main{#1393165}

--

wpt-commits: cebbe649f588028ec15cf314acc425d77f45da81
wpt-pr: 49576

UltraBlame original commit: d511b3e7274c52c45e83b17623bdf496b4543b7b
---
 .../pseudo-classes-after-part.html            |  1 -
 ...en-closed-pseudo.html => open-pseudo.html} | 20 +++----------------
 ...-light-dismiss-invalidation.tentative.html |  8 ++++----
 3 files changed, 7 insertions(+), 22 deletions(-)
 rename testing/web-platform/tests/css/selectors/{open-closed-pseudo.html => open-pseudo.html} (63%)

diff --git a/testing/web-platform/tests/css/css-shadow-parts/pseudo-classes-after-part.html b/testing/web-platform/tests/css/css-shadow-parts/pseudo-classes-after-part.html
index 60292d27a71c..873df7713987 100644
--- a/testing/web-platform/tests/css/css-shadow-parts/pseudo-classes-after-part.html
+++ b/testing/web-platform/tests/css/css-shadow-parts/pseudo-classes-after-part.html
@@ -15,7 +15,6 @@
 test_valid_selector("::part(mypart):any-link");
 test_valid_selector("::part(mypart):autofill");
 test_valid_selector("::part(mypart):checked");
-test_valid_selector("::part(mypart):closed");
 test_valid_selector("::part(mypart):default");
 test_valid_selector("::part(mypart):defined");
 test_valid_selector("::part(mypart):dir(ltr)");
diff --git a/testing/web-platform/tests/css/selectors/open-closed-pseudo.html b/testing/web-platform/tests/css/selectors/open-pseudo.html
similarity index 63%
rename from testing/web-platform/tests/css/selectors/open-closed-pseudo.html
rename to testing/web-platform/tests/css/selectors/open-pseudo.html
index 9301fd2f4162..0f3b9f3c3796 100644
--- a/testing/web-platform/tests/css/selectors/open-closed-pseudo.html
+++ b/testing/web-platform/tests/css/selectors/open-pseudo.html
@@ -17,53 +17,39 @@
 <script>
 test(() => {
   const dialog = document.querySelector('dialog');
-  assert_true(dialog.matches(':closed'),
-    ':closed should match when the dialog is closed.');
   assert_false(dialog.matches(':open'),
     ':open should not match when the dialog is closed.');
 
   dialog.show();
-  assert_false(dialog.matches(':closed'),
-    ':closed should not match after dialog.open().');
   assert_true(dialog.matches(':open'),
     ':open should match after dialog.open().');
 
   dialog.close();
   dialog.showModal();
-  assert_false(dialog.matches(':closed'),
-    ':closed should not match after dialog.showModal().');
   assert_true(dialog.matches(':open'),
     ':open should match after dialog.showModal().');
 
   dialog.close();
-}, 'The dialog element should support :open and :closed.');
+}, 'The dialog element should support :open.');
 
 test(() => {
   const details = document.querySelector('details');
-  assert_true(details.matches(':closed'),
-    ':closed should match when the details is closed.');
   assert_false(details.matches(':open'),
     ':open should not match when the details is closed.');
 
   details.open = true;
-  assert_false(details.matches(':closed'),
-    ':closed should not match when the details is open.');
   assert_true(details.matches(':open'),
     ':open should match when the details is open.');
-}, 'The details element should support :open and :closed.');
+}, 'The details element should support :open.');
 
 promise_test(async () => {
   const select = document.querySelector('select');
-  assert_true(select.matches(':closed'),
-    ':closed should match when the select is closed.');
   assert_false(select.matches(':open'),
     ':open should not match when the select is closed.');
 
   await test_driver.click(select);
   await new Promise(requestAnimationFrame);
-  assert_false(select.matches(':closed'),
-    ':closed should not match when the select is open.');
   assert_true(select.matches(':open'),
     ':open should match when the select is open.');
-}, 'The select element should support :open and :closed.');
+}, 'The select element should support :open.');
 </script>
diff --git a/testing/web-platform/tests/html/semantics/forms/the-select-element/select-pseudo-light-dismiss-invalidation.tentative.html b/testing/web-platform/tests/html/semantics/forms/the-select-element/select-pseudo-light-dismiss-invalidation.tentative.html
index c9631d92e125..749fd8cf2758 100644
--- a/testing/web-platform/tests/html/semantics/forms/the-select-element/select-pseudo-light-dismiss-invalidation.tentative.html
+++ b/testing/web-platform/tests/html/semantics/forms/the-select-element/select-pseudo-light-dismiss-invalidation.tentative.html
@@ -14,7 +14,7 @@
 select {
   background-color: rgb(0, 0, 255);
 }
-select:closed {
+select:not(:open) {
   background-color: rgb(0, 255, 0);
 }
 select:open {
@@ -29,7 +29,7 @@
 <script>
 promise_test(async () => {
   assert_equals(getComputedStyle(select).backgroundColor, 'rgb(0, 255, 0)',
-    'The select should match :closed at the start of the test.');
+    'The select should match :not(:open) at the start of the test.');
 
   await test_driver.click(select);
   assert_equals(getComputedStyle(select).backgroundColor, 'rgb(255, 0, 0)',
@@ -37,7 +37,7 @@
 
   await test_driver.click(opttwo);
   assert_equals(getComputedStyle(select).backgroundColor, 'rgb(0, 255, 0)',
-    'The select should match :closed after clicking an option.');
+    'The select should match :not(:open) after clicking an option.');
 
   await test_driver.click(select);
   assert_equals(getComputedStyle(select).backgroundColor, 'rgb(255, 0, 0)',
@@ -45,6 +45,6 @@
 
   await test_driver.click(button);
   assert_equals(getComputedStyle(select).backgroundColor, 'rgb(0, 255, 0)',
-    'The select should match :closed after light dismiss.');
+    'The select should match :not(:open) after light dismiss.');
 }, 'select should not match :open when light dismissed.');
 </script>