Bug 1878207 [wpt PR 44370] - Always call superclass in HTMLLabelElement::DefaultEventHandler, a=testonly

josepharhar · moz-wptsync-bot · commit 2d02f30a63bd · 2024-02-13T08:17:48.000Z
Automatic update from web-platform-tests Always call superclass in HTMLLabelElement::DefaultEventHandler HTMLLabelElement::DefaultEventHandler does not currently call up to HTMLElement::DefaultEventHandler in many cases. This patch makes sure that HTMLLabelElement always calls up to HTMLElement, which makes <label>s in <button>s work. WebKit's HTMLLabelElement already does this in all branches of HTMLLabelElement::defaultEventHandler since 2017: WebKit/WebKit@fec9c1d WebKit/WebKit@b9bad31 Fixed: 1523168 Change-Id: Ib9d0a63befbb930b9595bcc89fb5955050c97b2f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5260750 Commit-Queue: Joey Arhar <jarhar@chromium.org> Reviewed-by: Mason Freed <masonf@chromium.org> Cr-Commit-Position: refs/heads/main@{#1257674} -- wpt-commits: 3c00d743109ae3d06e874e56dd9287f3ded06f44 wpt-pr: 44370
diff --git a/testing/web-platform/tests/html/semantics/popovers/label-in-invoker.html b/testing/web-platform/tests/html/semantics/popovers/label-in-invoker.html
@@ -0,0 +1,23 @@
+<!DOCTYPE html>
+<link rel=author href="mailto:jarhar@chromium.org">
+<link rel=help href="https://bugs.chromium.org/p/chromium/issues/detail?id=1523168">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="/resources/testdriver.js"></script>
+<script src="/resources/testdriver-vendor.js"></script>
+
+<button popovertarget=mypopover>
+  <label>label</label>
+</button>
+<div id=mypopover popover=auto>popover</div>
+
+<script>
+promise_test(async() => {
+  const label = document.querySelector('label');
+  assert_false(mypopover.matches(':popover-open'),
+    'Popover should be closed at the start of the test.');
+  await test_driver.click(label);
+  assert_true(mypopover.matches(':popover-open'),
+    'The popover should be opened by clicking on the label.');
+}, 'Buttons with popovertarget should invoke targets even if there is a label in the button.');
+</script>
