903682 - [DeviceStorage] Add an extension for locked music files.

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect)

Description

[:Harald Kirschner :digitarald]

Encrypted MP3 as new type generated by the Forward Lock app and played back by the patches Music app.

Via Fabrice: Gecko needs to know about mapping the extension to a mime type and the mime type needs to be added to known music files for device-storage.

Comment 1

[Jason Smith [:jsmith]]

This doesn't need to be security-sensitive

Comment 2

[Jason Smith [:jsmith]]

Sounds like we would need to modify toolkit/content/devicestorage.properties to include mp3x then.

Comment 3

[David Flanagan [:djf]]

As noted in the parent bug, I think we should use a different filename extension than "mp3x" because we might need to work with locked media formats that are not mp3s. But we can't just add ".fl" to the end of the filename because then we wouldn't be able to distinguish locked music from locked images (though I'm hoping we never have to deal with locked images".

A google search indicates that the extension ".lcka" is not in use. We could use that for any "locked audio".  Then, if necessary we have .lcki for images and .lckv for videos.

Comment 4

[David Flanagan [:djf]]

Editing the summary, nominating for 1.2, and assigning to Dave.

Comment 5

[Dave Hylands [:dhylands]]

Attachment: Add .lcka extension as a device storage audio extension

Comment 6

[David Flanagan [:djf]]

Unchecking the confidential box. We can implement this in the open. If we need to discuss anything confidential, we can do that in the parent bug 887071.

Comment 7

[David Flanagan [:djf]]

Comment on attachment 791015 [details] [diff] [review]
Add .lcka extension as a device storage audio extension

Review of attachment 791015 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks, Dave!

Comment 8

[Dave Hylands [:dhylands]]

https://hg.mozilla.org/integration/b2g-inbound/rev/a4d600b4d8ba

Comment 9

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/a4d600b4d8ba