Created attachment 601302 [details] [diff] [review]
No any reason to expose them to a script. Scripts should deal with with DOM stuffs instead.
Comment on attachment 601302 [details] [diff] [review]
>diff --git a/accessible/public/nsIAccessible.idl b/accessible/public/nsIAccessible.idl
>@@ -54,17 +54,17 @@ interface nsIAccessibleRelation;
> * accessibility APIs like MSAA and ATK. Contains the sum of what's needed
> * to support IAccessible as well as ATK's generic accessibility objects.
> * Can also be used by in-process accessibility clients to get information
> * about objects in the accessible tree. The accessible tree is a subset of
> * nodes in the DOM tree -- such as documents, focusable elements and text.
> * Mozilla creates the implementations of nsIAccessible on demand.
> * See http://www.mozilla.org/projects/ui/accessibility for more information.
> interface nsIAccessible : nsISupports
> nsRefPtr<nsAccessible> acc(do_QueryObject(this));
>- if (!acc)
>+ if (!acc || acc->IsDefunct())
> return E_FAIL;
if that qi fails something has gone horribly wrong, so I'm not sure you need the null check but I doubt it hurts.
https://developer.mozilla.org/en/nsIAccessNode and https://developer.mozilla.org/en/nsIAccessible should be updated (nsIAccessNode was removed, all methods were moved to nsIAccessible)