382878 - Remove old state copying hack for link descendants

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Aaron Leventhal]

A descendant of a link gets extra accessible states from the link itself. This is necessary for the states LINKED and TRAVERSED but nothing else.

We should remove the extra state copying because it's not a useful hack.

Comment 1

[Aaron Leventhal]

Attachment: Remove extra code

Comment 2

[alexander :surkov (:asurkov)]

It looks this code has been added partially in bug 115330. Why are these changes reversed?

Comment 3

[Aaron Leventhal]

Surkov, I didn't understand MSAA as well back then.

There are some differences between how IE and Firefox deal with MSAA for links. In IE's MSAA, focus events go to the text child of the link, which is a bit strange. In Firefox the focus goes to the link itself.

At one point GW Micro asked me to make sure that LINKED and TRAVERSED get copied to descendants of links. When I fixed bug 115330 I wanted to do what IE does and copy all the states just to be sure we got it working correctly. But it does not make sense to copy all states to children.

Comment 4

[alexander :surkov (:asurkov)]

Comment on attachment 266940 [details] [diff] [review]
Remove extra code

ok, r=me