Closed
Bug 294073
Opened 19 years ago
Closed 19 years ago
ISimpleDOMNode uniqueID not returning values that match childID's
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: aaronlev)
Details
(Keywords: access)
Attachments
(1 file)
1.70 KB,
patch
|
timeless
:
review+
bzbarsky
:
superreview+
mkaply
:
approval1.8b2+
|
Details | Diff | Splinter Review |
We're using the old method of calculating unique ID's in get_nodeInfo(). We should just be using GetUniqueID() which does all the work.
Assignee | ||
Comment 1•19 years ago
|
||
Attachment #183541 -
Flags: superreview?(bzbarsky)
Attachment #183541 -
Flags: review?(timeless)
Comment 2•19 years ago
|
||
Comment on attachment 183541 [details] [diff] [review] Reuse GetUniqueID() method, and negate value in the same way that other code already does for event child ID's OK, I guess, though after NS_PTR_TO_INT32 it's no longer unique, in general.
Attachment #183541 -
Flags: superreview?(bzbarsky) → superreview+
Attachment #183541 -
Flags: review?(timeless) → review+
Assignee | ||
Updated•19 years ago
|
Attachment #183541 -
Flags: approval-l10n?
Assignee | ||
Comment 3•19 years ago
|
||
(In reply to comment #2) > (From update of attachment 183541 [details] [diff] [review] [edit]) > OK, I guess, though after NS_PTR_TO_INT32 it's no longer unique, in general. > It's the same as we use elsewhere in our MSAA code because there is no support for MSAA on 64 bit platforms.
Assignee | ||
Updated•19 years ago
|
Attachment #183541 -
Flags: approval-l10n? → approval1.8b2?
Comment 4•19 years ago
|
||
Comment on attachment 183541 [details] [diff] [review] Reuse GetUniqueID() method, and negate value in the same way that other code already does for event child ID's a=mkaply
Attachment #183541 -
Flags: approval1.8b2? → approval1.8b2+
Assignee | ||
Comment 5•19 years ago
|
||
Checking in accessible/src/msaa/nsAccessNodeWrap.cpp; /cvsroot/mozilla/accessible/src/msaa/nsAccessNodeWrap.cpp,v <-- nsAccessNodeWrap.cpp new revision: 1.16; previous revision: 1.15 done
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•