[SelectMenu] Ensure that option parts are focusable.
Since only <option> elements are allowed as <selectmenu> option
parts[1], there is no need to change the tabIndex attribute anymore and
HTMLOptionElement::SupportsFocus is updated to ensure that option parts
are focusable.
[1]: http://crrev.com/c/3251854
Bug: 1121840
Change-Id: Ib1d90a4f22adb61c9a7267d5bc78692491a5a82c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3252809
Commit-Queue: Ionel Popescu <iopopesc@microsoft.com>
Reviewed-by: Dan Clark <daniec@microsoft.com>
Reviewed-by: Mason Freed <masonf@chromium.org>
Cr-Commit-Position: refs/heads/main@{#937147}
diff --git a/html/semantics/forms/the-selectmenu-element/selectmenu-option-focusable.tentative.html b/html/semantics/forms/the-selectmenu-element/selectmenu-option-focusable.tentative.html
new file mode 100644
index 0000000..bcfc1b9
--- /dev/null
+++ b/html/semantics/forms/the-selectmenu-element/selectmenu-option-focusable.tentative.html
@@ -0,0 +1,49 @@
+<!DOCTYPE html>
+<html lang="en">
+<title>HTMLSelectMenuElement Test: option facusable</title>
+<link rel="author" title="Ionel Popescu" href="mailto:iopopesc@microsoft.com">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="/resources/testdriver.js"></script>
+<script src="/resources/testdriver-actions.js"></script>
+<script src="/resources/testdriver-vendor.js"></script>
+
+<selectmenu id="selectmenu0">
+ <option>one</option>
+ <option id="selectmenu0-option2">two</option>
+ <option>three</option>
+</selectmenu>
+
+<script>
+// See https://w3c.github.io/webdriver/#keyboard-actions
+const KEY_CODE_MAP = {
+ 'Enter': '\uE007',
+ 'Space': '\uE00D',
+ 'ArrowUp': '\uE013',
+ 'ArrowDown': '\uE015'
+};
+
+function clickOn(element) {
+ const actions = new test_driver.Actions();
+ return actions.pointerMove(0, 0, {origin: element})
+ .pointerDown({button: actions.ButtonType.LEFT})
+ .pointerUp({button: actions.ButtonType.LEFT})
+ .send();
+}
+
+promise_test(async t => {
+ const selectMenu = document.querySelector("#selectmenu0");
+ assert_false(selectMenu.open, "selectmenu should not be initially open");
+
+ await clickOn(selectMenu);
+ assert_true(selectMenu.open);
+ assert_equals(selectMenu.value, "one");
+
+ const option2 = document.querySelector('#selectmenu0-option2');
+ option2.focus();
+ assert_equals(document.activeElement, option2);
+
+ await test_driver.send_keys(selectMenu, KEY_CODE_MAP.Enter);
+ assert_equals(selectMenu.value, "two");
+}, "Validate <option> is focusable when is a descendant of <selectmenu>");
+</script>
\ No newline at end of file