Bug 1927802 [wpt PR 48854] - Throw exception for popovers/dialogs in non-active documents, a=testonly

marco-c · marco-c · commit b2a03712ee88 · 2024-11-01T02:26:51.000Z
Automatic update from web-platform-tests Throw exception for popovers/dialogs in non-active documents See discussion at: whatwg/html#10659 and spec PR at: whatwg/html#10705 Web-facing change PSA: https://groups.google.com/a/chromium.org/g/blink-dev/c/jRFiIIkXv_k/m/jnPTfg8WBgAJ Fixed: 373684393 Change-Id: I50e400ee526775f915f006865301fff2f04016b4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5943740 Reviewed-by: Domenic Denicola <domenicchromium.org> Auto-Submit: Mason Freed <masonfchromium.org> Commit-Queue: Domenic Denicola <domenicchromium.org> Cr-Commit-Position: refs/heads/main{#1375681} -- wpt-commits: 0b99ee5e8c799f9fbf7d1550ef72fec8bdb45760 wpt-pr: 48854 UltraBlame original commit: ac6cc55b745eb0d675d453266bde353b7d335402
diff --git a/testing/web-platform/tests/html/semantics/interactive-elements/the-dialog-element/dialog-active-document.html b/testing/web-platform/tests/html/semantics/interactive-elements/the-dialog-element/dialog-active-document.html
@@ -0,0 +1,16 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<link rel="author" href="mailto:masonf@chromium.org">
+<link rel=help href="https://github.com/whatwg/html/pull/10705">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+
+<script>
+  test(() => {
+    const doc = document.implementation.createHTMLDocument();
+    const dialog = doc.createElement('dialog');
+    doc.body.appendChild(dialog);
+    assert_throws_dom('InvalidStateError',() => dialog.showModal());
+    assert_false(dialog.matches('[open]'));
+  },'showModal should throw when the document isn\'t active');
+</script>
diff --git a/testing/web-platform/tests/html/semantics/interactive-elements/the-dialog-element/dialog-showModal.html b/testing/web-platform/tests/html/semantics/interactive-elements/the-dialog-element/dialog-showModal.html
@@ -181,8 +181,7 @@
     doc.body.appendChild(d11);
     this.add_cleanup(() => document.body.append(d11));
     assert_false(d11.open);
-    d11.showModal();
-    assert_true(d11.open);
-    this.add_cleanup(() => d11.close());
-  }, "Although the document is not attached to any pages, showModal() should execute as normal.");
+    assert_throws_dom("INVALID_STATE_ERR", () => d11.showModal());
+    assert_false(d11.open);
+  }, "When the document is not attached to any pages, showModal() should throw.");
 </script>
diff --git a/testing/web-platform/tests/html/semantics/popovers/popover-active-document.html b/testing/web-platform/tests/html/semantics/popovers/popover-active-document.html
@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<link rel="author" href="mailto:masonf@chromium.org">
+<link rel=help href="https://github.com/whatwg/html/pull/10705">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+
+<script>
+  test(() => {
+    const doc = document.implementation.createHTMLDocument();
+    const popover = doc.createElement('div');
+    popover.setAttribute('popover','');
+    doc.body.appendChild(popover);
+    assert_throws_dom('InvalidStateError',() => popover.showPopover());
+    assert_false(popover.matches(':open'));
+  },'showPopover should throw when the document isn\'t active');
+</script>
