826847 - [music] music app dies if there are high res mp4 video on the sdcard

Status: RESOLVED FIXED

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

Description

[Inder]

Put a high res mp4 video in the /sdcard/Movies folder and apparently music app tries to parse it and dies due to low-memory pressure event.

Sometimes I have seen warning from mediadb just before the low-memory memory-pressure event (see below) and sometimes it just dies without any message due to the event.

/GeckoConsole(  413): [JavaScript Warning: "Media resource blob:9427ce8f-713e-4e03-84a0-a70fda6a1f86 could not be decoded." {file: "app://music.example.com/index.html" line: 0}]
E/GeckoConsole(  413): Content JS WARN at app://music.example.com/shared/js/mediadb.js:1305 in metadataError: MediaDB: error parsing metadata for Movies/1280x720_24p_WP.MP4 : Unplayable music file
I/GonkMemoryPressure(  391): Dispatching low-memory memory-pressure event
I/GonkMemoryPressure(  125): Dispatching low-memory memory-pressure event
I/GonkMemoryPressure(  382): Dispatching low-memory memory-pressure event
I/GonkMemoryPressure(  343): Dispatching low-memory memory-pressure event
I/GonkMemoryPressure(  413): Dispatching low-memory memory-pressure event
I/GonkMemoryPressure(  648): Dispatching low-memory memory-pressure event
I/Gecko   (  125): -*- QCContentHelper_QC_B2G: receiveMessage: 'child-process-shutdown' arrived from content process

Comment 1

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Does your build have the fix for bug 824099?  I think this is a dup.

Comment 2

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

bug 826024 fixed : 2013-01-06 16:43:58 PST
https://hg.mozilla.org/mozilla-central/rev/013dae0ca576

Comment 3

[Joe Cheng [:jcheng] (please needinfo)]

Triage: BB+, P1, C4 - blocking 808607

Comment 4

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

99.9% sure this was fixed by bug 824099.  Please reopen if not.

Comment 5

[Inder]

Sorry for the delay. Verified and don't see the issue anymore so as Chris pointed out must have been fixed by bug 824099.