Closed Bug 883786 Opened 7 years ago Closed 4 years ago

[Music] Support parsing WAV metadata

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: dkuo, Unassigned)

References

Details

(Whiteboard: [MR1.2] [FT: Media Recording, Sprint] [MEDIA_TRIAGED])

For v1.2 we are going to support wav in Music app, besides adding *.wav to /toolkit/content/devicestorage.properties, we also need to enhance the metadata parser to be able to parse metadata from wav files.
Blocks: 873389
Depends on: 894778
Blocks: 894853
Assignee: nobody → dkuo
Whiteboard: [MR1.2]
blocking-b2g: --- → koi+
Whiteboard: [MR1.2] → [MR1.2] [FT: Media Recording, Sprint]
QA Contact: pyang
I just tested and found music(1) can list the .wav sound file in playlist.
After talking to PM. It is not necessary to parse WAV format in v1.2 now. For now, gecko already support WAV format to play the music. Change to koi? and remove the block for 894853 (Core music player to parse WAV file format).
No longer blocks: 894853
blocking-b2g: koi+ → koi?
blocking-b2g: koi? → ---
Whiteboard: [MR1.2] [FT: Media Recording, Sprint] → [MR1.2] [FT: Media Recording, Sprint] [MEDIA_TRIAGED]
Blocks: 1065858
Assignee: dkuo → nobody
Maybe we should WONTFIX this. Especially since we already support FLAC, WAV is just a bad choice to use for music. I don't have an issue with letting the music app *play* WAV files (especially in the open activity), but I think the maintenance burden of a metadata format for a file type that we almost never see isn't worth it.
Whiteboard: [MR1.2] [FT: Media Recording, Sprint] [MEDIA_TRIAGED] → [MR1.2] [FT: Media Recording, Sprint] [MEDIA_TRIAGED][wontfix?]
After discussing this with the other music developers, we've agreed that we should WONTFIX this, at least for the time being. In the future, it might be worth revisiting this, however.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
Whiteboard: [MR1.2] [FT: Media Recording, Sprint] [MEDIA_TRIAGED][wontfix?] → [MR1.2] [FT: Media Recording, Sprint] [MEDIA_TRIAGED]
You need to log in before you can comment on or make changes to this bug.