827337 - [Volume] - Unlock screen sound unaffected by mute system volume

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: Gaia::System, defect)

Description

[Aaron Train [:aaronmt]]

Currently when one mutes the device system volume and attempts to, at a further time, unlock the device (with lock-sound enabled), the device will still emit a high-volume sound-clip.

Device unlock is unaffected by system audio mute.

--
Unagi-b2g18 (01/07)

Comment 1

[Lukas Blakk [:lsblakk] use ?needinfo]

Batch edit: Bugs marked status-b2g18: affected after 2/13 branching of v1.0.1 are now also status-b2g18-v1.0.1: affected

Comment 2

[Mihai Cirlanaru [:mihai][:mcirlanaru]]

From what I noticed in lockscreen.js, the unlock sound is not registered with any audio channel type (cf. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/lockscreen.js#L563) and this might be the first issue in adjusting its volume.

Additionally, might be the case that the fix for bug 819842 will solve this, as long as the audio channel type is correctly set (to 'notification' I assume).

Alive, can you confirm on this? Bug 837477 seems to be within the same scope (i.e. making use of the correct audio channel type -- 'notification') and its fix was merged already.

Comment 3

[(inactive after 6/18) Alive Kuo [:alive]]

(In reply to Mihai Cirlanaru [:mcirlanaru] from comment #2)
> From what I noticed in lockscreen.js, the unlock sound is not registered
> with any audio channel type (cf.
> https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/lockscreen.
> js#L563) and this might be the first issue in adjusting its volume.

The default audio channel for audio element is normal, however the default channel we adjust using volume rockers is notification. I would say this is by design, but yes, it's not perfect so we're having many sound bugs relevant to silent the phone but not silent the channel you want it to be muted.

> 
> Additionally, might be the case that the fix for bug 819842 will solve this,
> as long as the audio channel type is correctly set (to 'notification' I
> assume).

I don't think the unlock sound is a notification.
So the workaround is just bug 819842, you're right. The strategy is to silent "notification/ringer" and "content/normal" at the same time.

Comment 4

[Mihai Cirlanaru [:mihai][:mcirlanaru]]

This was solved by the patch for bug 819842, marking it as RESOLVED > WORKSFORME.