Closed
Bug 425693
Opened 16 years ago
Closed 16 years ago
Remove all "isEmpty then return false" portions from MSAA methods.
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9
People
(Reporter: MarcoZ, Assigned: MarcoZ)
References
Details
(Keywords: access)
Attachments
(1 file)
2.25 KB,
patch
|
surkov
:
review+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
But leave the IA2 ones intact. This is to better guarantee interoperability with older screen readers that don't use IA2 for Firefox 3 yet, such as JAWS 8.
Assignee | ||
Comment 1•16 years ago
|
||
There are a few more impls in other files, but none of them have this construct. All others are IA2 and should remain in place to comply with spec.
Attachment #312280 -
Flags: review?(aaronleventhal)
Assignee | ||
Comment 2•16 years ago
|
||
Comment on attachment 312280 [details] [diff] [review] Remove the "if isempty then return false" blocks from MSAA related methods. Forgot that Aaron is on the definitive move this weekend...
Attachment #312280 -
Flags: review?(aaronleventhal) → review?(surkov.alexander)
Comment 3•16 years ago
|
||
Comment on attachment 312280 [details] [diff] [review] Remove the "if isempty then return false" blocks from MSAA related methods. r=me
Attachment #312280 -
Flags: review?(surkov.alexander)
Attachment #312280 -
Flags: review+
Attachment #312280 -
Flags: approval1.9?
Comment 4•16 years ago
|
||
Comment on attachment 312280 [details] [diff] [review] Remove the "if isempty then return false" blocks from MSAA related methods. a=beltzner
Attachment #312280 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Comment 5•16 years ago
|
||
/cvsroot/mozilla/accessible/src/msaa/nsAccessibleWrap.cpp,v <-- nsAccessibleWrap.cpp new revision: 1.117; previous revision: 1.116 done
You need to log in
before you can comment on or make changes to this bug.
Description
•