From df4d8d7f2fbc7d63f4d26ffdf216b6d5caef55c6 Mon Sep 17 00:00:00 2001
From: Dominic Farolino <dom@chromium.org>
Date: Sat, 2 Mar 2024 22:41:57 +0000
Subject: [PATCH] Bug 1881960 [wpt PR 44774] - DOM: Add WPTs for removing
 steps, a=testonly

Automatic update from web-platform-tests
DOM: Add WPTs for removing steps

This CL adds WPTs for removing steps, to ensure that script is never
executed during the removing steps hook [1], which are invoked
synchronously during per-node removal (that is, they are not batched,
but run this way per-node).

This CL:
 1. Augments the existing iframe removing steps WPT, to include
    `pagehide` events. At the time of writing this, all browsers deviate
    from the HTML Standard, and fail this test.
 2. Adds new tests for `<input>` and `<button>` to assert that when they
    are focused and subsequently removed from the DOM, the `blur` event
    is *not* fired, per the HTML Standard. Chromium fails this test,
    while Safari and Firefox pass it.

[1]: https://dom.spec.whatwg.org/#concept-node-remove-ext

Bug: 40150299
Change-Id: I0aa43755ccac8ea432056800c5c4167d8e4460e8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5323158
Commit-Queue: Dominic Farolino <dom@chromium.org>
Reviewed-by: Mason Freed <masonf@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1266801}

--

wpt-commits: 2c931feab8ac744f9bbe3e9550cc2fe85831fc47
wpt-pr: 44774
---
 .../insertion-removing-steps/blur-event.window.js     | 19 +++++++++++++++++++
 .../insertion-removing-steps-iframe.window.js         | 11 +++++++++++
 2 files changed, 30 insertions(+)
 create mode 100644 testing/web-platform/tests/dom/nodes/insertion-removing-steps/blur-event.window.js

diff --git a/testing/web-platform/tests/dom/nodes/insertion-removing-steps/blur-event.window.js b/testing/web-platform/tests/dom/nodes/insertion-removing-steps/blur-event.window.js
new file mode 100644
index 000000000000..4c8cd85cbf54
--- /dev/null
+++ b/testing/web-platform/tests/dom/nodes/insertion-removing-steps/blur-event.window.js
@@ -0,0 +1,19 @@
+test(() => {
+  const input = document.body.appendChild(document.createElement('input'));
+  input.focus();
+
+  let blurCalled = false;
+  input.onblur = e => blurCalled = true;
+  input.remove();
+  assert_false(blurCalled, "Blur event was not fired");
+}, "<input> element does not fire blur event upon DOM removal");
+
+test(() => {
+  const button = document.body.appendChild(document.createElement('button'));
+  button.focus();
+
+  let blurCalled = false;
+  button.onblur = e => blurCalled = true;
+  button.remove();
+  assert_false(blurCalled, "Blur event was not fired");
+}, "<button> element does not fire blur event upon DOM removal");
diff --git a/testing/web-platform/tests/dom/nodes/insertion-removing-steps/insertion-removing-steps-iframe.window.js b/testing/web-platform/tests/dom/nodes/insertion-removing-steps/insertion-removing-steps-iframe.window.js
index a10610f4677b..60c2bec0c8aa 100644
--- a/testing/web-platform/tests/dom/nodes/insertion-removing-steps/insertion-removing-steps-iframe.window.js
+++ b/testing/web-platform/tests/dom/nodes/insertion-removing-steps/insertion-removing-steps-iframe.window.js
@@ -109,6 +109,15 @@ function runRemovalTest(removal_method) {
     t.add_cleanup(() => removalObserver.disconnect());
 
     let iframe1UnloadFired = false, iframe2UnloadFired = false;
+    let iframe1PagehideFired = false, iframe2PagehideFired = false;
+    iframe1.contentWindow.addEventListener('pagehide', e => {
+      assert_false(iframe1UnloadFired, "iframe1 pagehide fires before unload");
+      iframe1PagehideFired = true;
+    });
+    iframe2.contentWindow.addEventListener('pagehide', e => {
+      assert_false(iframe2UnloadFired, "iframe2 pagehide fires before unload");
+      iframe2PagehideFired = true;
+    });
     iframe1.contentWindow.addEventListener('unload', e => iframe1UnloadFired = true);
     iframe2.contentWindow.addEventListener('unload', e => iframe2UnloadFired = true);
 
@@ -126,6 +135,8 @@ function runRemovalTest(removal_method) {
       div.innerHTML = '';
     }
 
+    assert_false(iframe1PagehideFired, "iframe1 pagehide did not fire");
+    assert_false(iframe2PagehideFired, "iframe2 pagehide did not fire");
     assert_false(iframe1UnloadFired, "iframe1 unload did not fire");
     assert_false(iframe2UnloadFired, "iframe2 unload did not fire");
 
-- 
2.11.4.GIT