Closed Bug 1181982 Opened 9 years ago Closed 9 years ago

[B2G] Manage the sound of the attention window

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:2.5+, b2g-master fixed)

VERIFIED FIXED
FxOS-S3 (24Jul)
blocking-b2g 2.5+
Tracking Status
b2g-master --- fixed

People

(Reporter: evanxd, Assigned: evanxd)

References

Details

(Keywords: regression, smoketest, Whiteboard: [dogfood-blocker])

Attachments

(1 file)

STR: 1. Set alarm in Clock app. 2. Play music in Music app. 3. Alarm is ringing. Excepted result: Alarm is playing, but music is not. Actual result: Alarm and music are playing at same time.
Looks like it is a gecko bug related with [1]. After applying the [2] patch and do the STR, you could see the below log. ``` I/GeckoConsole( 4261): Content JS LOG: bug 1181982: Music registers audio channels I/GeckoConsole( 4261): Content JS LOG: bug 1181982: audioChannels: [object BrowserElementAudioChannel],[object BrowserElementAudioChannel] I/GeckoConsole( 4261): Content JS LOG: bug 1181982: Clock registers audio channels I/GeckoConsole( 4261): Content JS LOG: bug 1181982: audioChannels: [object BrowserElementAudioChannel],[object BrowserElementAudioChannel] I/GeckoConsole( 4261): Content JS LOG: bug 1181982: clock.gaiamobile.org registers audio channels ``` The alarm attention window cannot get the `allowedAudioChannels`. Then the Audio Channel Service[3] cannot manage it. That's the root cause of this bug. [1]: https://bugzilla.mozilla.org/show_bug.cgi?id=1183301#c9 [2]: https://github.com/evanxd/gaia/commit/f38de9dfec23929ab4cd22ff733c67911676484d [3]: https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/audio_channel_service.js
Depends on: 1183301
Assignee: evanxd → nobody
Assignee: nobody → evanxd
Discussed with Kan-Ru and Alastor offline, we need to wait for mozbrowserloadend to get allowedAudioChannels from browser element.
Patch is ready for review. But need a reviewer.
Comment on attachment 8634629 [details] [gaia] evanxd:bug-1181982 > mozilla-b2g:master Hi Dominic, Could you help to review the patch? Thanks.
Attachment #8634629 - Flags: review?(dominic.kuo)
Comment on attachment 8634629 [details] [gaia] evanxd:bug-1181982 > mozilla-b2g:master Looks like we have to wait for mozbrowserloadend then can get correct allowedAudioChannels for app windows, and the patch does the right thing so looks good to me.
Attachment #8634629 - Flags: review?(dominic.kuo) → review+
Dominic, thanks for the review. But the patch causes some Gij failures, go fixing the tests.
Blocks: 1183710
Blocks: 1180618
No longer blocks: 1183710
No longer blocks: 1180618
This bug can solve the problem that we can't manage the sound of the attention window. For example, the alarm starts.
Summary: Alarm and Music are playing at same time. → [B2G] Manage the sound of the attention window
Carrying over the flags from dupe (bug 1186050)
blocking-b2g: --- → 2.5+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [dogfood-blocker]
This issue is verified fixed on the latest Spark 2.5 build. Alarms properly pause playing music before audibly sounding themselves, then un-pause the music when the alarm is dismissed. Environmental Variables: Device: Aries 2.5 Build ID: 20150724113133 Gaia: db299103dd17a873cba4ff88e62ec173bf80fe62 Gecko: 9bca608ab65a Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd Version: 42.0a1 (2.5) Firmware Version: D5803_23.1.A.1.28_NCB.ftf User Agent: Mozilla/5.0 (Mobile; rv:42.0) Gecko/42.0 Firefox/42.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(onelson)
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(onelson)
Target Milestone: --- → FxOS-S3 (24Jul)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: