395840 - accExplorer craches

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

Attachment: patch

accExplorer craches when invokes accHitTest and the method fails to find an accessible. According MSDN (http://msdn2.microsoft.com/en-us/library/ms696127.aspx ) we return bad COM error for this case.

Comment 1

[alexander :surkov (:asurkov)]

Comment on attachment 280544 [details] [diff] [review]
patch


>       if (!domNode) {
>         // Has already been shut down
>         pvarChild->vt = VT_EMPTY;
>-        return E_FAIL;
>+        return S_FALSE;
>       }

Though probably here I should fail still. Right?

Comment 2

[Aaron Leventhal]

Right. Reverse that change and r=me

Comment 3

[alexander :surkov (:asurkov)]

checked in with reversed change.