Implement setSequentialFocusStartingPoint

This is proposed here: whatwg/html#5326 Change-Id: I2c0c0ac578180ce5e1cf327fb65c6450a7d49f60 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4866726 Reviewed-by: Di Zhang <dizhangg@chromium.org> Commit-Queue: Joey Arhar <jarhar@chromium.org> Cr-Commit-Position: refs/heads/main@{#1197956}
chromium · Sep 18, 2023 · c098515 · c098515
1 parent 1ddf0a5
commit c098515
Show file tree

Hide file tree

Showing 4 changed files with 36 additions and 0 deletions.
diff --git a/third_party/blink/renderer/core/dom/document.idl b/third_party/blink/renderer/core/dom/document.idl
@@ -219,6 +219,10 @@ typedef (HTMLScriptElement or SVGScriptElement) HTMLOrSVGScriptElement;
     attribute EventHandler onresume;
     attribute EventHandler onsearch;
     attribute EventHandler onvisibilitychange;
+
+    // Proposed setter for focus navigation starting point
+    // https://github.com/whatwg/html/issues/5326
+    [RuntimeEnabled=SetSequentialFocusStartingPoint, ImplementedAs=SetSequentialFocusNavigationStartingPoint] void setSequentialFocusStartingPoint(Element element);
 };
 
 Document includes GlobalEventHandlers;

diff --git a/third_party/blink/renderer/platform/runtime_enabled_features.json5 b/third_party/blink/renderer/platform/runtime_enabled_features.json5
@@ -3361,6 +3361,10 @@
       origin_trial_feature_name: "ServiceWorkerStaticRouter",
       public: true,
     },
+    {
+      name: "SetSequentialFocusStartingPoint",
+      status: "experimental",
+    },
     {
       name: "SharedArrayBuffer",
       base_feature: "none",

diff --git a/..._tests/external/wpt/html/interaction/focus/setSequentialFocusStartingPoint.tentative.html b/..._tests/external/wpt/html/interaction/focus/setSequentialFocusStartingPoint.tentative.html
@@ -0,0 +1,27 @@
+<!DOCTYPE html>
+<link rel=author href="mailto:jarhar@chromium.org">
+<link rel=help href="https://github.com/whatwg/html/issues/5326">
+<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 id=b1>b1</button>
+<div id=d1>d1</div>
+<button id=b2>b2</button>
+
+<script>
+const tabKey = '\uE004';
+promise_test(async () => {
+  assert_equals(document.activeElement, document.body,
+    'Focus should initially be set on the body element.');
+
+  document.setSequentialFocusStartingPoint(d1);
+  assert_equals(document.activeElement, document.body,
+    'Calling setSequentialFocusStartingPoint should not change the focused element.');
+
+  await test_driver.send_keys(document.activeElement, tabKey);
+  assert_equals(document.activeElement, b2,
+    'Pressing tab should focus the next button after the sequential focus starting point.');
+}, 'document.setSequentialFocusStartingPoint should set the sequential focus starting point on any element.');
+</script>
diff --git a/third_party/blink/web_tests/webexposed/global-interface-listing-expected.txt b/third_party/blink/web_tests/webexposed/global-interface-listing-expected.txt
@@ -2040,6 +2040,7 @@ interface Document : Node
     method replaceChildren
     method requestStorageAccess
     method requestStorageAccessFor
+    method setSequentialFocusStartingPoint
     method startViewTransition
     method webkitCancelFullScreen
     method webkitExitFullscreen