462983 - always calculate extra states

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

1) we will do this eventually, see bug 344674
2) I need this to fix the bug 434707 because extra states should be passed into GetARIAState and they must be calculated at that point

Comment 1

[Aaron Leventhal]

I wonder if this will make perf worse for some ATs.

Maybe not for newer ones though, which are the ones we care about.

Comment 2

[alexander :surkov (:asurkov)]

(In reply to comment #1)
> I wonder if this will make perf worse for some ATs.
> 
> Maybe not for newer ones though, which are the ones we care about.

Now states affect on extra states. bug 434707 requires to make reverse dependence. So we need to compute extra states to finish states computing. If you care about performance then we shouldn't calculate all states together if we want to get one state and therefore we should have separate methods for every state (or something like).

Comment 3

[Aaron Leventhal]

Okay.

Comment 4

[Trevor Saunders (:tbsaunde)]

We know always calculate all states, just don't allways give all of them to js.

Comment 5

[alexander :surkov (:asurkov)]

(In reply to comment #4)
> We know always calculate all states, just don't allways give all of them to
> js.

When you mark fixed then please point what is a fix. 

Fixed by bug 634218.