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

RESOLVED WONTFIX

Status

()

Core
Disability Access APIs
RESOLVED WONTFIX
7 years ago
7 years ago

People

(Reporter: MarcoZ, Assigned: davidb)

Tracking

Trunk
x86
Windows 7
Points:
---

Firefox Tracking Flags

(blocking2.0 final+)

Details

(Whiteboard: [softblocker][patch needs vendor testing])

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

7 years ago
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

7 years ago
David, wanna take it?
Assignee: nobody → bolterbugz
(Reporter)

Comment 2

7 years ago
The module name is OsmHooks.dll.
(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.
blocking2.0: --- → final+
Whiteboard: [softblocker]
(Reporter)

Comment 5

7 years ago
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.
Good catch - I was sloppy.
Attachment #506728 - Flags: review?(marco.zehe)
Created attachment 506732 [details] [diff] [review]
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/dbolter@mozilla.com-359a76bcefdc
Attachment #506728 - Attachment is obsolete: true
Attachment #506732 - Flags: review?(marco.zehe)
(Reporter)

Comment 8

7 years ago
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.
Whiteboard: [softblocker] → [softblocker][patch needs vendor testing]
(Reporter)

Comment 9

7 years ago
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.
Attachment #506732 - Flags: review?(marco.zehe)
(Reporter)

Updated

7 years ago
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → WONTFIX
Depends on: 591874
No longer depends on: 591874
You need to log in before you can comment on or make changes to this bug.