1351097 - Unreferenced resource://gre/modules/accessibility/AccessFu.jsm file

Status: NEW

(Core :: Disability Access APIs, defect, P5)

Description

[Florian Quèze [:florian]]

My test from bug 1349005 indicates that resource://gre/modules/accessibility/AccessFu.jsm isn't referenced by any file shipped with Firefox desktop. Searching in mozilla-central shows that it's referenced by mobile/android/chrome/content/browser.js

Should this file be shipped only on android then?

Comment 1

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

The owner of accessfu is on leave. I'd support not shipping on desktop until we want it there for reasons. Yura are you aware of any gotchas?

Comment 2

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

The only other use on desktop was within Eitan's screen reader emulator add-on, which is likely broken and out of date. I am fine to not ship in Desktop until we have a usecase for it there.

Comment 3

[Eitan Isaacson [:eeejay]]

I think we should not ship this in Desktop.

OTOH, it is useful to test this with desktop mochitests. Can we not bundle it, but still have it around?

Comment 4

[Kris Maglione [:kmag]]

(In reply to Eitan Isaacson [:eeejay] from comment #3)
> I think we should not ship this in Desktop.
> 
> OTOH, it is useful to test this with desktop mochitests. Can we not bundle
> it, but still have it around?

Yes, it can be moved to TESTING_JS_MODULES and accessed from resource://testing-common/