867448 - [Music] Need to be able to save from the "open" activity window

Status: RESOLVED FIXED

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

Description

[Corey Frang [:gnarf]]

To copy some text from bug 840056:

The audio file is opened by the media player, while the sound is played in the music player, the user has the option to "save" that content, if the user selects that option, a banner indicating the success/failure of the operation is shown when completed.

Comment 1

[David Flanagan [:djf]]

The fix for this bug will probably resemble the fix for bug 867231.  If you take this bug, please see the work I've done in that one.  In particular, my pull request in that bug includes useful utilities in shared/js/device_storage_utils.js  You'll almost certainly want to use those for this bug as well.

Comment 2

[David Flanagan [:djf]]

This bug is blocked on a gecko crash in bug 868625. When that is fixed, I can probably take this one. But I haven't taken it yet in case someone else wants it.

Comment 3

[Dietrich Ayala (:dietrich)]

does this cover audio and video, or just audio?

Comment 4

[Dominic Kuo [:dkuo]]

I think I should be the someone else so I am taking it.

Comment 5

[Dominic Kuo [:dkuo]]

The one for video is filed in Bug 869292, this one will cover audio only.

Comment 6

[Dietrich Ayala (:dietrich)]

Blocks a blocker.

Comment 7

[Dominic Kuo [:dkuo]]

Attachment: add a save option to music app open activity

David,

This is the patch that implements the save funtion in open activity, I followed how gallery/video did and modified some parts base on the currect open acticity of music app. The major one is the workaround of bug 848723 casue it seems fixed to me so I removed it as well. And I also modified the metadata.js so our audio parser will be able to parse both file and blob without errors, that's becasue I heard that browser app might support save media files from web page and probably use open/view activity to achieve this.

Comment 8

[David Flanagan [:djf]]

Comment on attachment 747484 [details]
add a save option to music app open activity

Dominic,

This is a good patch, but I'm giving r- because one of the uitests doesn't work right (it won't save without a .ogg extension on the filename).

There is also an unsed title variable that can be removed.

Because the MMS team wants this quickly, I'll modify your patch to make those two minor changes and land that.

Comment 9

[David Flanagan [:djf]]

Attachment: minor changes to Dominic's patch

Attaching a link to a new PR that fixes the merge commits and a couple of trivial bugs in Dominic's patch so that it is ready to land and giving an r+

Comment 10

[David Flanagan [:djf]]

Landed on master: https://github.com/mozilla-b2g/gaia/commit/c00f94deb7325b024f37ca7ee0e7c51417ef2c87

Comment 11

[David Flanagan [:djf]]

Uplifted to v1-train: https://github.com/mozilla-b2g/gaia/commit/8aedc50576243652f33e25aa28306d1cb5cd700a