Open Bug 2032751 Opened 3 months ago Updated 1 month ago

tab.audible is undefined on Android

Categories

(WebExtensions :: Android, defect, P5)

defect

Tracking

(Not tracked)

People

(Reporter: robwu, Unassigned)

References

Details

On mobile, the audible property of a tab is computed from nativeTab.playingAudio:
https://searchfox.org/firefox-main/rev/23974e2d947e31e4ae42ae2758a4416c9a6d8671/mobile/shared/components/extensions/ext-android.js#321

That code originated from Fennec (old source), but does not work at all on Fenix.

bug 1962876 refactors the mechanism that reveals the audibility of a tab. I think that the desktop and Android implementations can become similar at that point.

Note: there is a report stating that desktop's currently implemented behavior is incorrect, see bug 1667585.

This is also dead code (there is nothing that registers the listener): https://searchfox.org/firefox-main/rev/0ae2fcae6a70566ab9e13e401480387b34911bad/mobile/shared/components/extensions/ext-tabs.js#178-185

once support for audible is re-introduced, an alternative needs to be implemented to make sure that tabs.onUpdated fires when the audible property changes.

See Also: → 2032928
Severity: -- → S4
Priority: -- → P5
See Also: → 1962876, 2046657
You need to log in before you can comment on or make changes to this bug.