1095698 - [Music] Tapping the play button, the song resume for one second in the notification screen

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: AudioChannel, defect)

Description

[pete siphantong [:Petes]]

Attachment: vm.txt

Description:
While a video is playing, pull down the notification screen and tap the play button, the resumes for one second

Setup Steps:
1) Flame device is set to 319mb

Repro Steps:
1) Update a Flame device to BuildID: 20141107001205
2) Go to Music app > play a song
3) Pause song > go to video app
4) Play video, pull down the notification screen
5) Press play


Actual:
song does not resume 

Expected:
song resume and the video in the background pauses

Flame 2.1 

Device: Flame 2.1 (319mb)(Kitkat Base)(Shallow Flash)
Build ID: 20141107001205
Gaia: 6295f6acfe91c6ae659712747dd2b9c8f51d0339
Gecko: 8c23b4f2ba29
Version: 34.0 (2.1)
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0


Repro frequency: 100%
See attached: video, logcat, https://www.youtube.com/watch?v=PpLBzmTP2LI

Comment 1

[pete siphantong [:Petes]]

This issue also reproduces on the Flame 2.2 and the Flame 2.0, the song resumes for one second.

Flame 2.2

Device: Flame 2.2 Master (319mb)(Kitkat Base)(Shallow Flash)
BuildID: 20141107040206
Gaia: 779f05fead3d009f6e7fe713ad0fea16b6f2fb31
Gecko: 64f4392d0bdc
Version: 36.0a1 (2.2 Master)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0


Flame 2.0 

Device: Flame 2.0 (319mb)(Kitkat Base)(Shallow Flash)
BuildID: 20141107000206
Gaia: d3e4da377ee448f9c25f908159480e867dfb13f3
Gecko: 9836e9d81357 
Version: 32.0 (2.0)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Comment 2

[Joshua Mitchell (Inactive)]

NI to Music owner for blocking decision

Comment 3

[No-Jun Park [:njpark]]

[Blocking Requested - why for this release]:
Breaks common use case scenario.  song should resume normally when selected by the user

Comment 4

[Jim Porter (:squib)]

(In reply to No-Jun Park [:njpark] from comment #3)
> [Blocking Requested - why for this release]:
> Breaks common use case scenario.  song should resume normally when selected
> by the user

I'm not sure that's the correct behavior. I'd have expected the play button to be disabled in the notification tree.

(I also disagree that this should be blocking, since it's not a regression. It *would* be nice to fix somehow, though.)

Comment 5

[Hema Koka [:hema]]

Jim or Hub: Lets get this fixed in master for 2.2...

Comment 6

[Jim Porter (:squib)]

Before we do that, I think we should check with UX to be sure we're doing the expected thing (see comment 4).

For what it's worth, I think doing as comment 0 suggests would be fairly difficult given how audio channels currently work. We'd probably have to bring the music app to the foreground, which would be pretty jarring. (Alternately, we could fix audio channels, but we've been talking about doing that for a while and haven't made much forward progress on it.)

Comment 7

[Jacqueline Savory [:jsavory] UX]

I think in this case, the video should pause when the utility tray is opened, instead of pausing when the music player is pressed. 

If we disable the play button, I feel the user may not understand why it is disabled and it may be a bit frustrating.

Comment 8

[Jim Porter (:squib)]

Does this still happen? There have been lots of AudioChannel changes since this bug was filed, and many of the issues have already been fixed.

Comment 9

[Carrie.Yang Leave from Mozilla]

Hi Jim,
   
    This bug can be repro on latest build of Flame KK v2.2(319mb& 512mb), but it can't be repro on latest build of Flame KK v2.5(512mb)&v2.6(512mb) and Aries KK v2.5&v2.6 by the STR in comment 0.

    When I try to repro this bug on 319mb memory of the latest Flame KK v2.5&v2.6 by the STR in comment 0, the Music app will be killed after lanuching Video app, so I can't check this bug on 319mb memory of Flame v2.5&v2.6.  Could you please help to confirm this? 

Thank you very much.

-------------------------------------------------------------------------------------------------------

Actual result:
On flame v2.2(319mb & 512mb), song does not resume.
Reproduce rate: 10/10.

On flame v2.5&v2.6(319mb), once Video app is opened, the Music process will be killed.
Reproduce rate: 0/10.

On flame v2.5&v2.6(512mb) and Aries v2.5&v2.6, song resumes and the video in the background pauses.
Reproduce rate: 0/10.

See attacthment: Flame_v2.2（319mb）_logcat_1544.txt, Affected_Flame_v2.2(319mb).3gp, Flame_v2.5(319mb).3gp and Unaffected_Flame_v2.5(512mb).3gp.

Device：Flame KK v2.2(319mb) (Affected) 
Build ID               20151108032507
Gaia Revision          885647d92208fb67574ced44004ab2f29d23cb45
Gaia Date              2015-10-07 13:05:24
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/e6ea91190b53
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20151108.065851
Firmware Date          Sun Nov  8 06:59:02 EST 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

Device：
Flame KK v2.5(319mb) (Blocked by the Music app is killed) 
Flame KK v2.5(512mb) (Unaffected) 
Build ID               20151108004501
Gaia Revision          577948202ae12154524a2bd2bd6d467838ad50b8
Gaia Date              2015-11-07 10:00:59
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/ae7b8b1fd9e1504347b938820f99d75058049386
Gecko Version          44.0a2
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20151108.045623
Firmware Date          Sun Nov  8 04:56:35 EST 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

Device：
Flame KK v2.6(319mb) (Blocked by the Music app is killed) 
Flame KK v2.6(512mb) (Unaffected)
Build ID               20151108150206
Gaia Revision          c3436122d678911d04b8f491724596116890ff9b
Gaia Date              2015-11-06 18:22:03
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/e2a910c048dc82fc3be53475f18e7f81f03e377b
Gecko Version          45.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20151108.183921
Firmware Date          Sun Nov  8 18:39:33 EST 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

Device：Aries KK v2.5 (Unaffected)
Build ID               20151107112205
Gaia Revision          577948202ae12154524a2bd2bd6d467838ad50b8
Gaia Date              2015-11-07 10:00:59
Gecko Revision         http://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/ae7b8b1fd9e1504347b938820f99d75058049386
Gecko Version          44.0a2
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20151107.104119
Firmware Date          Sat Nov  7 10:41:27 UTC 2015
Bootloader             s1

Device：Aries KK v2.6 (Unaffected) 
Build ID               20151107001102
Gaia Revision          c3436122d678911d04b8f491724596116890ff9b
Gaia Date              2015-11-06 18:22:03
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/e2a910c048dc82fc3be53475f18e7f81f03e377b
Gecko Version          45.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20151106.232924
Firmware Date          Fri Nov  6 23:29:32 UTC 2015
Bootloader             s1

Comment 10

[Carrie.Yang Leave from Mozilla]

Attachment: Flame_v2.2（319mb）_logcat_1544.txt

Comment 11

[Carrie.Yang Leave from Mozilla]

Attachment: Flame_v2.2(319mb).3gp

Comment 12

[Carrie.Yang Leave from Mozilla]

Attachment: Flame_v2.5(319mb).3gp

Comment 13

[Carrie.Yang Leave from Mozilla]

Attachment: Unaffected_Flame_v2.5(512mb).3gp

Comment 14

[Jim Porter (:squib)]

Thanks for checking! We can close this now, I think.