[webkit-changes] [WebKit/WebKit] b36557: AX: aria-hidden=false should be a synonym of undef...
Joshua Hoffman
noreply at github.com
Wed Oct 9 10:57:10 PDT 2024
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: b365574687fc38236c3abe1a5c1cc8e3fef0f712
https://github.com/WebKit/WebKit/commit/b365574687fc38236c3abe1a5c1cc8e3fef0f712
Author: Joshua Hoffman <jhoffman23 at apple.com>
Date: 2024-10-09 (Wed, 09 Oct 2024)
Changed paths:
A LayoutTests/accessibility/aria-hidden-false-ignored-expected.txt
A LayoutTests/accessibility/aria-hidden-false-ignored.html
R LayoutTests/accessibility/aria-hidden-false-works-in-subtrees.html
R LayoutTests/accessibility/aria-hidden-negates-no-visibility.html
A LayoutTests/accessibility/aria-hidden-subtree-expected.txt
A LayoutTests/accessibility/aria-hidden-subtree.html
M LayoutTests/accessibility/aria-modal-expected.txt
M LayoutTests/accessibility/aria-modal.html
R LayoutTests/accessibility/aria-visible-element-roles.html
M LayoutTests/accessibility/datetime/input-date-field-labels-and-value-changes.html
M Source/WebCore/accessibility/AXObjectCache.cpp
M Source/WebCore/accessibility/AXObjectCache.h
M Source/WebCore/accessibility/AccessibilityNodeObject.cpp
M Source/WebCore/accessibility/AccessibilityObject.cpp
M Source/WebCore/accessibility/AccessibilityRenderObject.cpp
Log Message:
-----------
AX: aria-hidden=false should be a synonym of undefined
https://bugs.webkit.org/show_bug.cgi?id=267150
rdar://120557669
Reviewed by Tyler Wilcock.
Per spec changes (see https://github.com/w3c/aria/pull/2090), we should treat `aria-hidden=false` as undefined.
This patch removes support for aria-hidden=false, updating places where we relied on the behaviors of isNodeARIAVisible
with the new behavior. For example, we need to check if a child node is focused before deciding whether to skip it if it is
aria-hidden (tested by accessibility/datetime/input-date-field-labels-and-value-changes.html).
Tests that explicitly validate aria-hidden false were removed, and a new test to check that we are ignoring this property
has been added.
* LayoutTests/accessibility/aria-hidden-false-ignored-expected.txt: Added.
* LayoutTests/accessibility/aria-hidden-false-ignored.html: Added.
* LayoutTests/accessibility/aria-hidden-false-works-in-subtrees.html: Removed.
* LayoutTests/accessibility/aria-hidden-negates-no-visibility.html: Removed.
* LayoutTests/accessibility/aria-hidden-subtree-expected.txt: Added.
* LayoutTests/accessibility/aria-hidden-subtree.html: Added.
* LayoutTests/accessibility/aria-modal-expected.txt:
* LayoutTests/accessibility/aria-modal.html:
* LayoutTests/accessibility/aria-visible-element-roles.html: Removed.
* LayoutTests/accessibility/datetime/input-date-field-labels-and-value-changes.html:
* Source/WebCore/accessibility/AXObjectCache.cpp:
(WebCore::AXObjectCache::modalElementHasAccessibleContent):
(WebCore::AXObjectCache::isNodeVisible const):
(WebCore::AXObjectCache::getOrCreate):
(WebCore::isNodeFocused):
(WebCore::isNodeAriaVisible): Deleted.
* Source/WebCore/accessibility/AXObjectCache.h:
* Source/WebCore/accessibility/AccessibilityNodeObject.cpp:
(WebCore::AccessibilityNodeObject::textUnderElement const):
* Source/WebCore/accessibility/AccessibilityObject.cpp:
(WebCore::AccessibilityObject::defaultObjectInclusion const):
* Source/WebCore/accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::addNodeOnlyChildren):
Canonical link: https://commits.webkit.org/284905@main
To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications
More information about the webkit-changes
mailing list