Closed Bug 566291 Opened 14 years ago Closed 14 years ago

DOMPointToHypertextOffset should return a start accessible as nsAccessible* return value

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: surkov, Assigned: surkov)

References

(Blocks 1 open bug)

Details

(Keywords: access)

Attachments

(1 file)

Attached patch patchSplinter Review
It's safe to abandon from nsresult as return value since DOMPointToHypertextOffset returns failure when something is really wrong, so we don't expect to get any failures. I'll leave warning to track possible failures.

Also, during this clean up I noticed that CreateTextChangeEventForNode() is wrong when it handles inaccessible node removal, I'll file separate bug on this.
Attachment #445663 - Flags: review?(bolterbugz)
Comment on attachment 445663 [details] [diff] [review]
patch

fine with me, r=me :)
Attachment #445663 - Flags: review?(bolterbugz) → review+
landed on 1.9.3 - http://hg.mozilla.org/mozilla-central/rev/f2df3688826e
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: