868945 - [Music] Music application doesn't get loaded, even though Music files are present in SDCard

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: Gaia::Music, defect)

Description

[Leo]

1. Title : Music player doesn't get loaded 
2. Precondition : have Music files on sd card
3. Tester's Action : Open Music App -> only a scanning icon is displayed , Player doesn't get loaded.
4. Detailed Symptom (ENG.) : When user taps on music player , the player doesn't get loaded .
5. Expected : Tapping on Music player should load all songs.
6. Gaia Version : # 61d7ab244db3e2174b22bfa6de3e3d69136b4904
7.Reproducibility: Y
	1)Frequency Rate : 100%
8.Comparison Results : 
	1)Model Comparing : 
9. Attached files: 
	1)Log : 
	2)Test Contents : 
	3)Video file :

Comment 1

[Partha]

Hi,

As per the code it is expected that DOMContentLoaded is triggered first and TabBar.init() is called even before the localized event occurs.

In some cases localized event is triggered even before the TabBar.init() is called, this is the reason why TabBar.playlistArray.localize() is not defined and Music application is not getting loaded.

dkuo,can you please share your opinion on this?

Comment 2

[Dominic Kuo [:dkuo]]

Partha,

I have removed listening to the DOMContentLoaded event and put all the init works in localized event in bug 849766, if this issue is casued by the race condition you mentioned, and that patch should also solve this problem.

Comment 3

[Leo]

When this race condition occurs, user can not use music application. Music application should be killed to use it again. Marking leo?.

Comment 4

[Wayne Chang [:wchang]]

Mark as dup per comment 2. Please retest once bug 849766 has been uplifted

Comment 5

[Dominic Kuo [:dkuo]]

Sorry everyone, I have reopen this because if the race condition which mentioned in Comment 1 also happens on v1.0.1, then this issue should be tef+ because the v1.0.1 branch doesn't have the patch of bug 849766, we have to create a new patch that specially for solving the race condition, so I am nominating this to tef+.

Comment 6

[Alex Keybl [:akeybl]]

QA - have you seen this in 1.0.1/unagi testing? If so, please specify how frequently or if there were specific conditions that caused this.

Leaning towards tef-

Comment 7

[mlevin]

Issue does not repro on latest Unagi.

Environmental  Variables:
Mozilla RIL
Unagi Build ID: 20130513070209
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/6e1dbf036d34
Gaia: 54dfb62735c81152eef74eb1768e2bf621c0a258

Removing QAWanted keyword.

Comment 8

[Marcia Knous [:marcia]]

On 1.0.1 branch, I was able to reproduce this with a game - see Bug 872246

Comment 9

[Alex Keybl [:akeybl]]

Based upon comment 7, not blocking. bug 872246 is a different issue that our partners have not brought up as blocking v1.0.1.

Comment 11

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Reproducible with steps from bug 879018

2) Launch "Music" app for the first time from the home screen
3) Tap and hold home button and then tap on "X" to kill the "Music" app
4) Open the music application for the second time

Comment 12

[Dominic Kuo [:dkuo]]

Thanks Naoki, by using today's pvt build, I was able to reproduce this issue as well, this should be a blocker and please mark it tef+, thanks. I will send a patch asap.

Comment 13

[Andrew Overholt [:overholt]]

(In reply to Naoki Hirata :nhirata (please use needinfo instead of cc) from comment #11)
> Reproducible with steps from bug 879018
> 
> 2) Launch "Music" app for the first time from the home screen
> 3) Tap and hold home button and then tap on "X" to kill the "Music" app
> 4) Open the music application for the second time

Since most users won't be doing the long press close thing, it's not terribly concerning.  However, what if the Music app is killed in a low memory situation; does the same problem occur?

Comment 15

[Alex Keybl [:akeybl]]

Is this a recent regression? If not, this is surely tef- at this point.

Comment 16

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Looks like this bug doesn't happen on 2.0 nor 1.1 unagi 
Just checked with 1.1 Leo in today's build.  

I think we can close this as WFM now.

Comment 17

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Gecko  http://hg.mozilla.org/releases/mozilla-b2g18/rev/d85aa34a49f2
Gaia   0198c8980b454deb965830a729d5951952daf010
BuildID 20130613070213
Version 18.0