From 0a8bcf69e861a5e07aaec6d86147ab439f9c46cb Mon Sep 17 00:00:00 2001 From: Mason Freed Date: Fri, 21 Apr 2023 21:40:11 +0000 Subject: [PATCH] Bug 1828556 [wpt PR 39589] - Reset the popover invoker in `showPopover()`, a=testonly Automatic update from web-platform-tests Reset the popover invoker in `showPopover()` As was pointed out in [1], if we don't reset invoker, it is possible to get into a state where a closed popover still has an invoker. The new test case in this CL shows that problem. [1] https://github.com/whatwg/html/issues/9152 Bug: 1307772 Change-Id: I1511ec8f0e9e0193f0c8b5ce834d20de9d8b46e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4435697 Reviewed-by: Joey Arhar Commit-Queue: Tommy Steimel Code-Coverage: Findit Auto-Submit: Mason Freed Reviewed-by: Tommy Steimel Cr-Commit-Position: refs/heads/main@{#1133290} -- wpt-commits: cc0704e3a1741d28d2bef73c6e84796b7c928849 wpt-pr: 39589 --- .../popovers/popover-invoker-reset.html | 34 +++++++++++++++++++ 1 file changed, 34 insertions(+) create mode 100644 testing/web-platform/tests/html/semantics/popovers/popover-invoker-reset.html diff --git a/testing/web-platform/tests/html/semantics/popovers/popover-invoker-reset.html b/testing/web-platform/tests/html/semantics/popovers/popover-invoker-reset.html new file mode 100644 index 0000000000000..4b1ac6b588f70 --- /dev/null +++ b/testing/web-platform/tests/html/semantics/popovers/popover-invoker-reset.html @@ -0,0 +1,34 @@ + + + + + + + + + + + + +
Popover 1 + +
+
Popover 2
+ +