1205377 - [Notifications]The "Playing song" icon doesn't appear in status bar after finishing a voice call

Status: RESOLVED DUPLICATE of bug 1199605

(Firefox OS Graveyard :: AudioChannel, defect, P4)

Description

[Max Ivanov[:MaxIvanov]]

Attachment: logs.txt

Description:
After ending a voice call a user is not notified in a status bar that a song is playing. 

Repro Steps:
1) Update a Flame to 20150916030229
2) Go to Music app and play any song
3) Press "home button" observe "Song is playing" icon in a notification bar
4) Start a voice call
5) Finish a voice call
6) Observe a status bar

Actual:
No "Song is playing" icon in a status bar


Expected:
"Song is playing" icon in a status bar

Environmental Variables:
Device: Flame 2.5
Build ID: 20150916030229
Gaia: 994ff1537c2d7ca4d1658806c50f3ceba1053f9b
Gecko: 3e8dde8f8c174cce2c0b65c951808f88e35d1875
Gonk: c4779d6da0f85894b1f78f0351b43f2949e8decd
Version: 43.0a1 (2.5)
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0

Repro frequency: (100 %)
See attached: (video clip, logcat)
https://youtu.be/4AHq8mSTfmU

Comment 1

[Max Ivanov[:MaxIvanov]]

This issue DOES occur on Flame 2.5 and Aries 2.5 and does NOT on Flame 2.2 
Result:"Song is playing" icon is not showed in a status bar

Device: Flame 2.5
Build ID: 20150916030229
Gaia: 994ff1537c2d7ca4d1658806c50f3ceba1053f9b
Gecko: 3e8dde8f8c174cce2c0b65c951808f88e35d1875
Gonk: c4779d6da0f85894b1f78f0351b43f2949e8decd
Version: 43.0a1 (2.5)
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0
-----------------------------------------------------------------

Device: Aries 2.5
Build ID: 20150916015546
Gaia: 994ff1537c2d7ca4d1658806c50f3ceba1053f9b
Gecko: 3e8dde8f8c174cce2c0b65c951808f88e35d1875
Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd
Version: 43.0a1 (2.5)
Firmware Version: D5803_23.1.A.1.28_NCB.ftf
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0

Comment 2

[KTucker [:KTucker][Inactive 3/4/2016]]

[Blocking Requested - why for this release]:

Regression that will be encountered by end users.

Comment 3

[Sxean Lee-David [:sxean]]

B2G inbound window: 

Last Working
Environmental Variables:
Device: Flame 2.5
BuildID: 20150914235936
Gaia: 01cf74daf8e1c1fbc945dd61ff0b2524f4b06b8e
Gecko: 64960a58c9857af68b994ae8f303e708205ac8f6
Gonk: c4779d6da0f85894b1f78f0351b43f2949e8decd
Version: 43.0a1 (2.5) 
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0

First Broken
Device: Flame 2.5
BuildID: 20150915002837
Gaia: 01cf74daf8e1c1fbc945dd61ff0b2524f4b06b8e
Gecko: 5a91d8cf566dc4879a7b7e10dd41fc349cfddf61
Gonk: c4779d6da0f85894b1f78f0351b43f2949e8decd
Version: 43.0a1 (2.5) 
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0


First broken Gecko Last working Gaia - Repro
Gaia: 01cf74daf8e1c1fbc945dd61ff0b2524f4b06b8e
Gecko: 5a91d8cf566dc4879a7b7e10dd41fc349cfddf61

Last working Gecko first broken Gaia - NoRepro
Gaia: 01cf74daf8e1c1fbc945dd61ff0b2524f4b06b8e
Gecko: 64960a58c9857af68b994ae8f303e708205ac8f6

Gecko Pushlog: 
http://hg.mozilla.org/integration/b2g-inbound/pushloghtml?fromchange=64960a58c9857af68b994ae8f303e708205ac8f6&tochange=5a91d8cf566dc4879a7b7e10dd41fc349cfddf61

This issue appears to be caused by changes made in bug # 1129882

Comment 4

[Jayme Mercado [:JMercado] (Inactive 3/4/2016)]

Alastor this is another issue that seems to have been caused by the changes for bug 1129882.  Can you please take a look?

Comment 5

[Alastor Wu [:alwu]]

Hi, Jim,
Do you know where is the code about handling the play icon in status bar? (In music app)
Thanks!

Comment 6

[Alastor Wu [:alwu]]

I found the possible root cause, that is we don't send the "audiochannelchanged" event to notify the sound manager when we resume the music. Therefore, we would get the wrong current audio channel.

Ex.
1. Play music
   - current channel = content
2. Start telephony, and pause music
   - current channel = telephony
3. Stop telephony, and resume music 
   - current channel = none (X)

---

Hi, Evan,
Could we send this event when resume audio?
Thanks!

Comment 7

[Jim Porter (:squib)]

I don't think we do anything special in the music app, except for setting the audio channel in the manifest.

Comment 8

[Evan Tseng [:evanxd]]

Hi Alastor,

The patch for Bug 1199605 will fix this.