539696 - reorg IEnumVariant implementation

Status: NEW

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

1) nsAccessible and AccessibleEnumerator implementation are very close and they could share the same implementation especially after we started to use array to store accessible children
2) two ATs can use IEnumVariant the same time and this may make them to get wrong children
3) there is a public copy constructor which should be obviously private (this was inspired by talk with Pete)

Comment 1

[alexander :surkov (:asurkov)]

Attachment: wip

the main problem I have met is how to unattach the enumvariant when its data are out of date for example when children are changed, accessible goes away, changed the number of selected children. Filing wip, probably once I'll get back.

Comment 2

[Trevor Saunders (:tbsaunde)]

Alex, still want to do something here? iirc you did a bunch of this stuff a while ago.

Comment 3

[alexander :surkov (:asurkov)]

(In reply to Trevor Saunders (:tbsaunde) from comment #2)
> Alex, still want to do something here? iirc you did a bunch of this stuff a
> while ago.

per bug description we can say it's fixed, however we still need to clean up AccessibleEnumerator class (perhaps move it to EnumVariant files, address 3d item, keep it updated when array is changed)