corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1946912: Properly expose read-only property to UIA for text fields, r=Jamie

Browse Source 
It was possible for <input> and <textarea> to incorrectly report the IsReadOnly
property as "true" to UIA, particularly when empty. There is no text leaf in
empty text inputs, so our TextLeafPoint would point to the text field itself.
Code that retrieves attributes for UIA didn't handle this case, which caused AT
issues, such as Narrator failing to switch out of scan mode automatically.

This revision fixes the issue by properly checking for text fields when figuring
out whether a TextLeafPoint's Accessible is read-only. It also adds a test to
verify the same.

Differential Revision: https://phabricator.services.mozilla.com/D237644
This commit is contained in:
Nathan LaPre
2025-02-13 01:45:57 +00:00
parent c71999aeb4
commit 739b849487
2 changed files with 27 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
accessible/windows/uia/UiaTextRange.cpp
View File

@@ -1307,15 +1307,20 @@ struct AttributeTraits<UIA_IsReadOnlyAttributeId> {
if (!aPoint.mAcc) {
return {};
}
// Check the parent of the leaf, since the leaf itself will never be
// editable, but the parent may. Check for both text fields and hypertexts,
// since we might have something like <input> or a contenteditable <span>.
Accessible* acc = aPoint.mAcc;
Accessible* parent = acc->Parent();
if (parent && parent->IsHyperText()) {
acc = parent;
} else {
return Some(true);
// If the TextLeafPoint we're dealing with is itself a hypertext, don't
// bother checking its parent since this is the Accessible we care about.
if (!acc->IsHyperText()) {
// Check the parent of the leaf, since the leaf itself will never be
// editable, but the parent may. Check for both text fields and
// hypertexts, since we might have something like <input> or a
// contenteditable <span>.
Accessible* parent = acc->Parent();
if (parent && parent->IsHyperText()) {
acc = parent;
} else {
return Some(true);
}
}
const uint64_t state = acc->State();
if (state & states::READONLY) {