560670 - nsIAccessibleText should use the text control specific selection functions instead of nsISelection

Status: NEW

(Core :: Disability Access APIs, defect)

Description

[(no longer active)]

Using the specialized selection APIs for text controls will initialize the editor beforehand.  Currently, nsIAccessibleText seems to be using the document selection APIs, which is both wrong, and will not initialize the editor.

We need to switch to using those APIs (selectionStart and selectionEnd properties) in our nsIAccessibleText implementation.

Comment 1

[alexander :surkov (:asurkov)]

Ehsan, what is specialized selection API?

Comment 2

[(no longer active)]

The following API:

nsITextContolFrame::SetSelectionStart
nsITextContolFrame::SetSelectionEnd
nsITextContolFrame::SetSelectionRange
nsITextContolFrame::GetSelectionRange

And also nsITextContolFrame::GetOwnedSelectionController if you need access to the selection controller used by the text control frame.

Comment 3

[(no longer active)]

Fixing this bug may be a simple task by the patch in bug 560665.  I suspect that if you also register nsEditorInitializerOnSelection on the SELECTION_ACCESSIBILITY selection type as well, you'd get a fix for free.  Of course someone on the a11y team needs to confirm this.

I'm setting a dependency for now.

Comment 4

[(no longer active)]

(In reply to comment #3)
> Fixing this bug may be a simple task by the patch in bug 560665.  I suspect
> that if you also register nsEditorInitializerOnSelection on the
> SELECTION_ACCESSIBILITY selection type as well, you'd get a fix for free.  Of
> course someone on the a11y team needs to confirm this.
> 
> I'm setting a dependency for now.

What I said here might not be a good answer, because that code listens to selections on the pres shell, and we really don't want to do that here, because the text box has its own selection controller.  But I think a similar approach in nsTextInputListener might work here.

Comment 5

[(no longer active)]

Although, comment 4 is only true for editor initialization.  The selection manipulations performed by nsIAccessibleText would still be on the wrong selection object.