Closed
Bug 351563
Opened 19 years ago
Closed 19 years ago
Accessible Wrap classes no longer used
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: surkov)
Details
(Keywords: access, regression)
Attachments
(1 file)
|
2.38 KB,
patch
|
aaronlev
:
review+
hwaara
:
review+
|
Details | Diff | Splinter Review |
Spinoff from bug 351549.
Platform-specific behavior exists in the Wrap classes. However, since bug 241015, we no longer create the appropriate Wrap classes.
Example:
http://lxr.mozilla.org/seamonkey/source/accessible/src/base/nsAccessibilityService.cpp#1360
Should create nsXULMenuitemAccessibleWrap, not an nsXULMenuitemAccessible.
If we don't, the GetName method is not overridden, and the correct string for menuitems is not exposed. (In MSAA the accessible name must have a tab character separating the menuitem label and the accelerator key).
|
Reporter | |
Updated•19 years ago
|
|
Assignee | |
Comment 1•19 years ago
|
||
Attachment #237091 -
Flags: review?(aaronleventhal)
|
|
Assignee | |
Updated•19 years ago
|
Attachment #237091 -
Flags: review?(hwaara)
|
|
Assignee | |
Updated•19 years ago
|
Status: NEW → ASSIGNED
|
||
Comment 2•19 years ago
|
||
Comment on attachment 237091 [details] [diff] [review]
patch
Compiles fine, and changes all of the wrap classes that are typedef'd in the headers, and created in the acc service.
Attachment #237091 -
Flags: review?(hwaara) → review+
|
|
Reporter | |
Updated•19 years ago
|
Attachment #237091 -
Flags: review?(aaronleventhal) → review+
|
|
||
Comment 3•19 years ago
|
||
Checked in for Alex to trunk.
Is this code needed on branch too?
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
|
|
Reporter | |
Comment 4•19 years ago
|
||
Not a bug on branch.
You need to log in
before you can comment on or make changes to this bug.
Description
•