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)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
People
(Reporter: surkov, Assigned: surkov)
References
(Blocks 1 open bug)
Details
(Keywords: access)
Attachments
(1 file)
18.69 KB,
patch
|
davidb
:
review+
|
Details | Diff | Splinter 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 1•14 years ago
|
||
Comment on attachment 445663 [details] [diff] [review] patch fine with me, r=me :)
Attachment #445663 -
Flags: review?(bolterbugz) → review+
Assignee | ||
Comment 2•14 years ago
|
||
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.
Description
•