Bug 1949254 [wpt PR 50808] - Improve testing dialog closedby and requestClose, a=testonly

Automatic update from web-platform-tests
Improve testing dialog closedby and requestClose

See this request:

whatwg/html#10983 (comment)

plus this comment:

https://chromium-review.googlesource.com/c/chromium/src/+/6245064/comments/e932d31a_c996d18e

This just adds more testing of dialog closedby and requestClose, to
make sure close watchers are still set up, the `:open` pseudo class
works properly, events get fired correctly, etc., when the dialog
`open` attribute is manually added and removed.

Bug: 376516550
Change-Id: Ibdee09c5f722db2ce97c64c38c39726c1503e1d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6253798
Auto-Submit: Mason Freed <masonfchromium.org>
Commit-Queue: Mason Freed <masonfchromium.org>
Reviewed-by: Joey Arhar <jarharchromium.org>
Reviewed-by: Luke <lwarlowigalia.com>
Cr-Commit-Position: refs/heads/main{#1422074}

--

wpt-commits: 57d6088fe0481e9662763d2147831219e692d3bc
wpt-pr: 50808

UltraBlame original commit: 9668c9902b5818811f24cb50228e71a8fa1ca08e

Author: marco-c

testing/web-platform/tests/html/semantics/interactive-elements/the-dialog-element/dialog-closedby-start-open.html

@@ -233,6 +233,19 @@
 open
 )
 ;
+assert_true
+(
+dialog
+.
+matches
+(
+'
+:
+open
+'
+)
+)
+;
 await
 new
 test_driver
@@ -252,6 +265,19 @@
 open
 )
 ;
+assert_false
+(
+dialog
+.
+matches
+(
+'
+:
+open
+'
+)
+)
+;
 dialog
 .
 showModal
@@ -265,6 +291,19 @@
 open
 )
 ;
+assert_true
+(
+dialog
+.
+matches
+(
+'
+:
+open
+'
+)
+)
+;
 await
 new
 test_driver
@@ -284,6 +323,19 @@
 open
 )
 ;
+assert_false
+(
+dialog
+.
+matches
+(
+'
+:
+open
+'
+)
+)
+;
 }
 Dialogs
 that
@@ -483,6 +535,19 @@
 open
 )
 ;
+assert_true
+(
+dialog
+.
+matches
+(
+'
+:
+open
+'
+)
+)
+;
 await
 new
 test_driver
@@ -508,6 +573,19 @@
 '
 )
 ;
+assert_false
+(
+dialog
+.
+matches
+(
+'
+:
+open
+'
+)
+)
+;
 }
 Opening
 and

testing/web-platform/tests/html/semantics/interactive-elements/the-dialog-element/dialog-closedby.html

@@ -378,6 +378,19 @@
 open
 )
 ;
+assert_false
+(
+dialog
+.
+matches
+(
+'
+:
+open
+'
+)
+)
+;
 if
 (
 modal
@@ -406,6 +419,19 @@
 open
 )
 ;
+assert_true
+(
+dialog
+.
+matches
+(
+'
+:
+open
+'
+)
+)
+;
 assert_equals
 (
 dialog
@@ -453,6 +479,19 @@
 open
 )
 ;
+assert_false
+(
+dialog
+.
+matches
+(
+'
+:
+open
+'
+)
+)
+;
 t
 .
 add_cleanup
@@ -514,6 +553,20 @@
 .
 open
 ;
+const
+respondsToEsc2
+=
+!
+dialog
+.
+matches
+(
+'
+:
+open
+'
+)
+;
 dialog
 .
 close
@@ -545,6 +598,20 @@
 .
 open
 ;
+const
+respondsToLightDismiss2
+=
+!
+dialog
+.
+matches
+(
+'
+:
+open
+'
+)
+;
 dialog
 .
 close
@@ -603,6 +670,22 @@
 ;
 assert_true
 (
+respondsToEsc2
+'
+Dialog
+should
+respond
+to
+ESC
+(
+:
+open
+)
+'
+)
+;
+assert_true
+(
 respondsToLightDismiss
 '
 Dialog
@@ -614,6 +697,23 @@
 '
 )
 ;
+assert_true
+(
+respondsToLightDismiss2
+'
+Dialog
+should
+respond
+to
+light
+dismiss
+(
+:
+open
+)
+'
+)
+;
 break
 ;
 case
@@ -633,6 +733,22 @@
 '
 )
 ;
+assert_true
+(
+respondsToEsc2
+'
+Dialog
+should
+respond
+to
+ESC
+(
+:
+open
+)
+'
+)
+;
 assert_false
 (
 respondsToLightDismiss
@@ -647,6 +763,24 @@
 '
 )
 ;
+assert_false
+(
+respondsToLightDismiss2
+'
+Dialog
+should
+NOT
+respond
+to
+light
+dismiss
+(
+:
+open
+)
+'
+)
+;
 break
 ;
 case
@@ -669,6 +803,23 @@
 ;
 assert_false
 (
+respondsToEsc2
+'
+Dialog
+should
+NOT
+respond
+to
+ESC
+(
+:
+open
+)
+'
+)
+;
+assert_false
+(
 respondsToLightDismiss
 '
 Dialog
@@ -681,6 +832,24 @@
 '
 )
 ;
+assert_false
+(
+respondsToLightDismiss2
+'
+Dialog
+should
+NOT
+respond
+to
+light
+dismiss
+(
+:
+open
+)
+'
+)
+;
 break
 ;
 case