628644 - Add the Baum Retec AG Cobra screen reader module to the window emulation whitelist

Status: RESOLVED WONTFIX

(Core :: Disability Access APIs, defect)

Description

[Marco Zehe (:MarcoZ)]

Baum Retec AG's screen reader named Cobra, a German screen reader, supports Firefox 3.6 and 4, and for some of their stuff relies on the Window hierarchy. To fix them up quickly, we should just add the module name they inject to our whitelist to turn on window hierarchy emulation. I'm waiting for the module name to be sent to me.

Comment 1

[alexander :surkov (:asurkov)]

David, wanna take it?

Comment 2

[Marco Zehe (:MarcoZ)]

The module name is OsmHooks.dll.

Comment 3

[David Bolter [:davidb] (NeedInfo me for attention)]

(In reply to comment #1)
> David, wanna take it?

Sure.

Hopefully it won't need tweaks.

By the way we should be pinging all the vendors we can to test with FF4 betas.

Comment 4

[David Bolter [:davidb] (NeedInfo me for attention)]

Attachment: simply adds module

http://ftp.mozilla.org/pub/mozilla.org/firefox/tryserver-builds/[e-mail]-78abac4f88dd

Comment 5

[Marco Zehe (:MarcoZ)]

Comment on attachment 506728 [details] [diff] [review]
simply adds module

>+const LPCWSTR kCobraModuleHandle = L"OsmHooks.dll";

Hmm the others are all without ".dll" and in lower case. Do you think we can apply the same rule here? For example, the JAWS module is called jhook.dll on the hard drive, but "jhook" is what gets returned I believe.

Comment 6

[David Bolter [:davidb] (NeedInfo me for attention)]

Good catch - I was sloppy.

Comment 7

[David Bolter [:davidb] (NeedInfo me for attention)]

Attachment: adds module name

Marco thanks for catching my sloppiness in the first patch. Here's the new build URL:
http://ftp.mozilla.org/pub/mozilla.org/firefox/tryserver-builds/[e-mail]-359a76bcefdc

Comment 8

[Marco Zehe (:MarcoZ)]

Comment on attachment 506732 [details] [diff] [review]
adds module name

OK, I'll send the try-server build to the developer and ask him to test as soon as it's ready before I r+. In principle this looks like a no-brainer, but I want to wait for testing from the vendor.

Comment 9

[Marco Zehe (:MarcoZ)]

Comment on attachment 506732 [details] [diff] [review]
adds module name

This bug is no longer needed, as per e-mail discussion with the Cobra developer. It doesn't solve their problem, in fact it would recreate a problem for them. Will close this bug as WONTFIX.