Closed Bug 425693 Opened 12 years ago Closed 12 years ago

Remove all "isEmpty then return false" portions from MSAA methods.

Categories

(Core :: Disability Access APIs, defect)

x86
Windows XP
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla1.9

People

(Reporter: MarcoZ, Assigned: MarcoZ)

References

Details

(Keywords: access)

Attachments

(1 file)

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.
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)
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 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 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+
Keywords: checkin-needed
/cvsroot/mozilla/accessible/src/msaa/nsAccessibleWrap.cpp,v  <--  nsAccessibleWrap.cpp
new revision: 1.117; previous revision: 1.116
done

Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Depends on: 438784
You need to log in before you can comment on or make changes to this bug.