Allow QueryInterface to IAccessible2 without crashing

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: aaronlev, Assigned: aaronlev)

Tracking

({access})

Trunk
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Because of the startup crashes in bug 376239 I had to back out these 2 lines from nsAccessibleWrap::QueryInterface() :
  //else if (IID_IAccessible2 == iid)
    //*ppv = NS_STATIC_CAST(IAccessible2*, this);

So the QI to IAccessible2 will no longer succeed.

We need to figure out what the mismatch is between versions of the screen readers and IAccessible2:
- Have they been revving the IID between versions of IAccessible2?
- Does the crash go away if we upgrade to the latest screen readers and copies of the IA2 IDL?
Surkov, you might want to check through all the IDL changes and see if there's anything else we need to do.

I've tested this with Window-Eyes 6.1 Beta 2D and it no longer crashes. On the other hand, Window-Eyes no longer works once we allow the QI to succeed. We'll have to recommend they turn IA2 support off by default for Firefox, at least for now until we know what we have. Otherwise Window-Eyes 6.1 users may not be able to use Firefox 3 at all.
Attachment #260871 - Flags: review?(surkov.alexander)
Comment on attachment 260871 [details] [diff] [review]
Update to the latest IDL and fix get_uniqueID() so it doesn't try to use *null

Thank you for the work. I'll put new version with minor changes.
Attachment #260871 - Flags: review?(surkov.alexander) → review+
Posted patch patch2 (obsolete) — Splinter Review
1) events introduced by not longer existing IA2 events are removed
2) fix role mapping
3) no longer existing methods are removed

I still have a crash but I guess it's because we use different versions of Window Eyes. IA2 guids should be really changed every time.
Attachment #260889 - Flags: review?(aaronleventhal)
Posted patch patch3Splinter Review
forgot to add new files
Attachment #260889 - Attachment is obsolete: true
Attachment #260890 - Flags: review?(aaronleventhal)
Attachment #260889 - Flags: review?(aaronleventhal)
Attachment #260890 - Flags: review?(aaronleventhal) → review+
checked in
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.