425213 - FOCUSABLE state not exposed for descendants of container with aria-activedescendant

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Aaron Leventhal]

Attachment: Expose FOCUSABLE if has role and id, and ancestor has aria-activedescendant

When aria-activedescendant is used, any descendant child of it with an ID is potentially focusable.

We should expose at least those descendants of the aria-activedescendant container with an ID and a role as being FOCUSABLE. I'm requiring a role in this algorithm because real HTML form controls are focusable anyway.

Testcase: http://www.mozilla.org/access/dhtml/listbox

Note: aria-activedescendant="" means that a descendant is potentially focused, but currently the container itself has focus.

Comment 1

[alexander :surkov (:asurkov)]

// ancestor has activedescendant property, this content could be activ

nit: line up this line

It's interesting to know how do you assume to deal with the bug 425125 which is the part of this one

Comment 2

[alexander :surkov (:asurkov)]

Comment on attachment 311821 [details] [diff] [review]
Expose FOCUSABLE if has role and id, and ancestor has aria-activedescendant

r=me

Comment 3

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 311821 [details] [diff] [review]
Expose FOCUSABLE if has role and id, and ancestor has aria-activedescendant

a=beltzner

Comment 4

[Marco Zehe (:MarcoZ)]

Requesting that this be checked in on Aaron's behalf. Aaron is away from the computer for another couple of days. Thanks!

Comment 5

[:Gijs (he/him)]

Merged to work after checkin for bug 425125, and checked in:
Checking in mozilla/accessible/src/base/nsAccessibilityService.cpp;
/cvsroot/mozilla/accessible/src/base/nsAccessibilityService.cpp,v  <--  nsAccessibilityService.cpp
new revision: 1.276; previous revision: 1.275
done
Checking in mozilla/accessible/src/base/nsAccessible.cpp;
/cvsroot/mozilla/accessible/src/base/nsAccessible.cpp,v  <--  nsAccessible.cpp
new revision: 1.368; previous revision: 1.367
done

Comment 6

[alexander :surkov (:asurkov)]

in-testsuite?