Closed
Bug 1083806
Opened 11 years ago
Closed 11 years ago
Remote Ring should use the ‘ringtone’ audio channel, not ‘content’
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wmathanaraj, Assigned: marta)
References
Details
Attachments
(1 file)
|
1.24 KB,
patch
|
freddy
:
feedback+
|
Details | Diff | Splinter Review |
Impact: The content channel has a lower playback priority and may not be audible when another app is playing something.
Fix: Use “ringer” (commands.js) (NB this requires the “audio-channel-ringer” permission)
We use the code from find my device, and it does use that content channel:
https://github.com/mozilla-b2g/gaia/blob/master/apps/findmydevice/js/commands.js
Flags: needinfo?(wmathanaraj)
| Reporter | ||
Comment 2•11 years ago
|
||
if it being not always fully audible is not an issue we dont need to make a change but given the feature set I think it makes sense to make that change so the audio is always noticeable.
Mozilla recommendation is to make the changes to ensure its covered against any situation.
Flags: needinfo?(wmathanaraj) → needinfo?(marta)
Assignee: bugzilla → marta
Status: NEW → ASSIGNED
Flags: needinfo?(marta) → needinfo?(fbraun)
Comment 4•11 years ago
|
||
Comment on attachment 8507715 [details] [diff] [review]
bug_1083806.patch
Review of attachment 8507715 [details] [diff] [review]:
-----------------------------------------------------------------
This patch looks alright to me
Attachment #8507715 -
Flags: feedback+
Updated•11 years ago
|
Flags: needinfo?(fbraun)
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•