917399 - [AccessFu] Remove mozContentEvent listener and listen for settings changes directly

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Eitan Isaacson [:eeejay]]

Currently we listen for ContentStart and then mozContentEvent in order to know if the accessibility.screenreader setting has been modified. This only works with the b2g chrome js, and thus not in the browser.

I would suggest we get the screen reader setting directly from the settings API.

Comment 1

[Eitan Isaacson [:eeejay]]

Attachment: Use settings API for enabling/disabling AccessFu.

Comment 2

[Yura Zenevich [:yzen] (please use "needinfo")]

Comment on attachment 806145 [details] [diff] [review]
Use settings API for enabling/disabling AccessFu.

Review of attachment 806145 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with a small change :)

::: accessible/src/jsat/AccessFu.jsm
@@ +39,5 @@
>        // Not on Android
> +      if (aWindow.navigator.mozSettings) {
> +        let lock = aWindow.navigator.mozSettings.createLock();
> +        let req = lock.get(SCREENREADER_SETTING);
> +        req.addEventListener('success', () => {

Better to use req.onsuccess = () => {...};

Comment 3

[Eitan Isaacson [:eeejay]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/10dbb333ca83

After a short discussion on IRC, we decided to keep addEventListener

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/10dbb333ca83

Comment 5

[Marco Zehe (:MarcoZ)]

Comment on attachment 806145 [details] [diff] [review]
Use settings API for enabling/disabling AccessFu.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): changed contentScript behavior causing screen reader to not fully load when activating it from Gaia's Developer settings panel.
User impact if declined: Developers could not actually test accessibility because parts of the screen reader would not load, like Exploring the screen would not work.
Testing completed (on m-c, etc.): Yes.
Risk to taking this patch (and alternatives if risky): None.
String or IDL/UUID changes made by this patch: None.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/c638f5733b14