Closed Bug 1165048 Opened 10 years ago Closed 10 years ago

Music playback is getting stopped and is not resumed when modem is restarted

Categories

(Core Graveyard :: Widget: Gonk, defect)

Product:
Core Graveyard
Component:
Widget: Gonk
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(blocking-b2g:2.2+, firefox39 wontfix, firefox40 wontfix, firefox41 fixed, b2g-v2.2 fixed, b2g-master fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla41
Project Flags:
blocking-b2g 2.2+
Tracking Flags:
Tracking Status
firefox39 --- wontfix
firefox40 --- wontfix
firefox41 --- fixed
b2g-v2.2 --- fixed
b2g-master --- fixed

People

(Reporter: diego, Assigned: diego)

References

Details

(Whiteboard: [caf priority: p2][CR 834802])

Attachments

(1 file)

Add permission to modify audio settings to system process
(deleted), patch
mwu
: review+
jocheng
: approval-mozilla-b2g37+
Details | Diff | Splinter Review
This is happening on a Lollipop prototype being used by CAF. When the modem is restarted on this device the audio media framework gets hosed and no more audio clips can be played until the phone is restarted. The reason is that in this device the gonk system process attempts to update audio settings. However, we currently only allow the radio and bluetooth processes to update audio settings [1]. This is easily fixed by granting audio settings permission to the system process too (AID_SYSTEM). [1] https://mxr.mozilla.org/mozilla-central/source/widget/gonk/GonkPermission.cpp#123
Blocks: CAF-v2.2-metabug
blocking-b2g: --- → 2.2?
Whiteboard: [CR 834802]
Whiteboard: [CR 834802] → [caf priority: p2][CR 834802]
Hsin-Yi, are you ok with Diego's suggestion to grant audio settings permission to System?
Flags: needinfo?(htsai)
(In reply to Dylan Oliver [:doliver] from comment #1) > Hsin-Yi, are you ok with Diego's suggestion to grant audio settings > permission to System? It looks fine to me, but Michael seems the right reviewer.
Flags: needinfo?(htsai) → needinfo?(mwu)
Steven, what do you think for this?
Flags: needinfo?(slee)
(In reply to Ken Chang[:ken] from comment #3) > Steven, what do you think for this? I think it's OK, but we need mwu's approval.
Flags: needinfo?(slee)
Flags: needinfo?(mwu)
Attachment #8607716 - Flags: review?(mwu)
Assignee: nobody → dwilson
Status: NEW → ASSIGNED
Attachment #8607716 - Flags: review?(mwu) → review+
Comment on attachment 8607716 [details] [diff] [review] Add permission to modify audio settings to system process NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings. [Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 976839 User impact if declined: Bluetooth will not recover gracefully after a bluetooth subsystem crash. Testing completed: Force crashed the bluetooth subsystem on Flame Kitkat and a Lollipop prototype device. My bluetooth headset reconnected correctly. Risk to taking this patch (and alternatives if risky): There are no obvious risks I can identify. String or UUID changes made by this patch: None
Attachment #8607716 - Flags: approval-mozilla-b2g37?
https://hg.mozilla.org/mozilla-central/rev/c92eb84dfd39
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox41: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
blocking-b2g: 2.2? → 2.2+
status-b2g-v2.2: --- → affected
status-b2g-master: --- → fixed
Attachment #8607716 - Flags: approval-mozilla-b2g37? → approval-mozilla-b2g37+
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: