1118148 - [FFOS2.0][Woodduck][Download]The audio play in browser cannot pause when play the downloads list audio.

Status: RESOLVED WORKSFORME

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

Description

[sync-1]

DEFECT DESCRIPTION:
 ->The audio play in browser cannot pause when play the downloads list audio.
 
 REPRODUCING PROCEDURES:
 ->1.Launch http://imps.tcl-ta.com/liuqiong/media/alca_mp3.html,touch the mp3 file to play in browser;
 ->2.When play audio,touch this audio to download;
 ->3.After download finished,drug down the status bar,touch the audio to play,we will listen two sounds---KO 
 (The download list audio and browser audio will play together)
 ->
 
 EXPECTED BEHAVIOUR:
 ->When play download list audio,the browser audio should pause.
 
 yujuan：07522611649（61649）
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
 
  REPRODUCING RATE:
 
  For FT PR, Please list reference mobile's behavior:

Comment 1

[sync-1]

Attachment: log

Comment 2

[sync-1]

Attachment: log

Comment 3

[Gregor Wagner [:gwagner]]

Can we get UX input here?

Comment 4

[Josh Cheng [:josh]]

Hi Norry,
qawanted for Woodduck 2.0M and Flame 2.0/2.1/2.2. Thanks!

Comment 5

[yifan [:yifan][:yliao]]

I'm not sure how to perform step 2:

->2.When play audio,touch this audio to download;

Could you please be more specific or provide a video? Thanks.

Comment 6

[Yeojin Chung [:YeojinC]]

This issue DOES reproduce on Flame 2.2, 2.1, 2.0, 188-1 base image(2.0).

Result: The browser audio does not pause. The browser player and the downloaded file play simultaneously.

Environmental Variables:
Device: Flame 2.2
BuildID: 20150107051358
Gaia: cd5c88c77debee57c87da79a3995bb34521fb14b
Gecko: 206205dd8bd1
Version: 37.0a1 (2.2) 
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0

Device: Flame 2.1
BuildID: 20150105144122
Gaia: b04a8cb7b2482e0a44e6702b48c42283a00b5b1e
Gecko: 99cea2c818f6
Version: 34.0 (2.1) 
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Device: Flame 2.0
BuildID: 20150106184227
Gaia: f76014fd2c7528493b90d759c68ec3070233d094
Gecko: 53ff92e647a0
Version: 32.0 (2.0) 
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0


Note: To download the file, press and hold the player > select "Save Audio".

Leaving qawanted for 2.0M

Comment 7

[sync-1]

Created an attachment (id=1105744)
 reproduce video

Comment 8

[sync-1]

Created an attachment (id=1105744)
 reproduce video

Comment 9

[sync-1]

Attachment: reproduce video

Created an attachment (id=1105744)
 reproduce video

Comment 10

[Shally Li（leave from Mozilla）]

I can repro this bug on Woodduck 2.0.
See Attchments:verify_v2.0M.mp4 and logcat_2237.txt.

Woodduck 2.0 build:
Gaia-Rev        
06b544f8a9ee03754dd3f5020f279cffa7a75804
Gecko-Rev       6b1e24a1cba1d5542bebb503ce05a1ec61ceb7d3
Build-ID        20150108050313
Version         32.0
Device-Name     jrdhz72_w_ff
FW-Release      4.4.2
FW-Incremental  1420664769
FW-Date         Thu Jan  8 05:06:39 CST 2015

Comment 11

[Shally Li（leave from Mozilla）]

Attachment: logcat_2237.txt

Comment 12

[Shally Li（leave from Mozilla）]

Attachment: verify_v2.0M.mp4

Comment 13

[yifan [:yifan][:yliao]]

In this situation the browser page audio channel is 'normal' and the music activity window audio channel is 'content'. Gecko handles the competition of audio channels. Josh could you please help to find support from platform? Thank you.

Comment 14

[Josh Cheng [:josh]]

Hi Randy,
Could you help to check this issue? Thank you!

Comment 15

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

Hi Domi, 
Is it the visible problem?

Comment 16

[Stephany Wilkes]

Flagging Francis here, and ni? Jacqueline as back-up while Rob and I are out (I am on PTO in Mexico so did not see this until now) and Francis is traveling.

Comment 17

[Josh Cheng [:josh]]

Hi Alastor,
Could you please help to check the problem? Thanks!

Comment 18

[Alastor Wu [:alwu]]

Hi, Josh,
I will check it later, and keep NI to trace this.

Comment 19

[Alastor Wu [:alwu]]

Hi, Josh,
This bug can be fixed after we complete the audio channel refactor.

The root cause of this problem is that we can't figure out which app is on the foreground. 
When we play the music from the download notification, this behavior doesn't change the web app into background. This makes both apps are visible, so the AudioChannelService doesn't know which one should be muted.

From the log, we can confirm this.

1) Play music from the web app
11090 11090 I Gecko   : DD | MEdiaElement init agent, winID = 4, visible = 1
11090 11090 I Gecko   : DD | HTMLMediaElement::CanPlayChanged, canplay = 0
11090 11090 I Gecko   : DD | AudioChannelAgent::NotifyAudioChannelStateChanged, visible = 1
11090 11090 I Gecko   : DD | HTMLMediaElement::CanPlayChanged, canplay = 0

2) Then play music from the download notification
11386 11386 I Gecko   : DD | MEdiaElement init agent, winID = 1, visible = 1
11090 11090 I Gecko   : DD | AudioChannelAgent::NotifyAudioChannelStateChanged, visible = 1
11386 11386 I Gecko   : DD | HTMLMediaElement::CanPlayChanged, canplay = 0
11090 11090 I Gecko   : DD | HTMLMediaElement::CanPlayChanged, canplay = 0
11090 11090 I Gecko   : DD | AudioChannelAgent::NotifyAudioChannelStateChanged, visible = 1
11090 11090 I Gecko   : DD | HTMLMediaElement::CanPlayChanged, canplay = 0
11386 11386 I Gecko   : DD | AudioChannelAgent::NotifyAudioChannelStateChanged, visible = 1
11386 11386 I Gecko   : DD | HTMLMediaElement::CanPlayChanged, canplay = 0

Comment 20

[Francis Djabri [:djabber]]

I agree with the expected behavior in the bug description.

Comment 21

[Josh Cheng [:josh]]

HI Alastor,
Thanks for the check. Do you think we should dup this one to bug 1113086 or bug 1113086 is just a pre-req?

Comment 22

[Alastor Wu [:alwu]]

Hi, Josh,
I think this is not suitable to mark as duplicate, maybe pre-req.

Comment 23

[Josh Cheng [:josh]]

Triage: 
Still WIP in 2.2 and non trivial. WOnt fix in 2.0/2.0M. 
Consider fix in 2.2

Comment 24

[Dominic Kuo [:dkuo]]

Alastor is correct and we need the new audio channel api to fix this.

Comment 25

[Bobby Chien]

[Blocking Requested - why for this release]: need to be fixed by new audiochannel in v3.0

Comment 26

[Mahendra Potharaju [:mahe]]

Alastor, Can you check if this can come in for 2.5 ?

Thanks

Comment 27

[Bobby Chien]

New audio channel is for 2.5 only. We should only verify this in 2.5 release. 

Per comment 23, we should also consider not to fix this in 2.1x and 2.2x. 

What do you think?

Comment 28

[Alastor Wu [:alwu]]

The link in description is broken, so I use the other way to verify it.
The result is that this issue doesn't exist in v2.5.

Test steps,
(1) Play audio from any website
(2) Download audio from https://goo.gl/O7FXft
(3) Play the downloaded audio from the notification bar

Result,
(4) The previous audio would be paused and playback the downloaded audio

Comment 29

[Josh Cheng [:josh]]

(In reply to Bobby Chien [:bchien] from comment #27)
> New audio channel is for 2.5 only. We should only verify this in 2.5
> release. 
> 
> Per comment 23, we should also consider not to fix this in 2.1x and 2.2x. 
> 
> What do you think?

Thanks Bobby, Agree wontfix for 2.1/2.2 until partner request.
Thanks!

Comment 30

[Ken Chang[:kenkai]]

According to Comment 28.