371750 - make nsIAccessible::keyBinding work for several actions

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

IAccessibleAction defines this method as

Returns an array of BSTRs describing one or more key bindings, if there
   are any, associated with the specified action.

   Note: the client allocates and passes in an array of pointers.  The server
   allocates the BSTRs and passes back one or more pointers to these BSTRs into
   the array of pointers allocated by the client.  The client is responsible 
   for deallocating the BSTRs.

   @param [in] actionIndex
    This index specifies the action of which to return the key bindings.
   @param [in] nMaxBinding
    The maximum number of keybindings requested, i.e. the size of the array 
	allocated to hold the results.
   @param [out] keyBinding
    An array of BSTRs, one for each key binding.
   @param [out] nBinding
    The actual number of key bindings returned.
  */
  [propget] HRESULT keyBinding
    (
     [in] long actionIndex,
     [in] long nMaxBinding,
     [out, size_is(,nMaxBinding), length_is(,*nBinding)] BSTR **keyBinding,
	 [out, retval] long *nBinding
    );

We should have something similiar for nsIAccessible.

Comment 1

[Aaron Leventhal]

The keybinding for the element in general is probably the same as the keybinding for action # 0

Action 0 corresponds to the "default action" in MSAA. The default action is what happens when you click on an element.

Comment 2

[alexander :surkov (:asurkov)]

Attachment: patch

Aaron, whom can I ask to review/check atk part?

Comment 3

[Aaron Leventhal]

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

Ginn, could you look this over? (Especially the ATK changes).

Comment 4

[Ginn Chen]

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

excellent

Comment 5

[alexander :surkov (:asurkov)]

Checked in on trunk