385444 - Simplify code that decides whether to fire SHOW/HIDE/REORDER

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Aaron Leventhal]

Right now we have 3 case:
1) Do not fire SHOW/HIDE/REORDER (doc not loaded and no accessibles yet)
2) Fire them (doc is loaded)
3) Sometimes fire them (doc partially loaded and some accessibles created)

We want to remove #3 because it makes the code larger and more confusing, and is an unncessary optimization. If there are any accessibles (other than the doc accessible) already created, lets just go ahead and fire SHOW/HIDE/REORDER.

Comment 1

[Aaron Leventhal]

Attachment: If content is not loaded and only doc accessible exists, exit early, otherwise run the full invalidation sequence

Comment 2

[alexander :surkov (:asurkov)]

Do we need GetAccessibleInParentChain() then? Can we merge it completely with GetParent()?

Comment 3

[Aaron Leventhal]

We need it, because it takes a DOM node which might not have an accessible.

Comment 4

[alexander :surkov (:asurkov)]

Comment on attachment 269358 [details] [diff] [review]
If content is not loaded and only doc accessible exists, exit early, otherwise run the full invalidation sequence

r=me