1062069 - [Ringtones] Some audios which list at My ringtones/alerts could not be set as default ringtone/alert

Status: VERIFIED FIXED

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

Description

[Alison Shiue]

Attachment: test_audios.zip

Gaia      fbb297c39aab5f17b179533d2a9a6c5166b2c197
Gecko     https://hg.mozilla.org/releases/mozilla-aurora/rev/fb5e796da813
BuildID   20140902160204
Version   34.0a2
Base image v123

STR:
1. Insert a SD card with some songs under /Notifications and /Ringtones folder
2. Go to Settings -> Sound
3. Under the "Tones" category, select "Manage Tones" 
4. Select "[AMR (audio)]test.amr" as default ringtone 
(please refer attachment test_audios.zip, the two audios have the same problem) 

Expect result:
The selected song become default ringtone successfully

Actual result:
Always show "Saving ringtone..." and could not set the audio as default ringtone (http://youtu.be/clVpvIXiL1c)

Comment 1

[Alison Shiue]

Attachment: saving_ringtone_issue.log

Comment 2

[Alison Shiue]

[Blocking Requested - why for this release]:
This problem would let user feel frustrated

Comment 3

[Hema Koka [:hema]]

Triage on 9/3

Blocking: User should be able to pick amr files to set as default ringtone as it is a supported format. Furthermore, the message shown is mis-leading; If we are unable to set a ringtone as default, UI should say so preferably with a reason and not just hang on "Saving Ringtone..."

Comment 4

[Hema Koka [:hema]]

Jim, please investigate this one.

Comment 5

[Jim Porter (:squib)]

It seems like this happens if you open the settings app, receive a call, and then try to change the ringtone. File formats don't seem to matter, and even built-in ringtones can break this. I get a lot of this in the logcat when it happens:

I/Gecko   (  291): 
I/Gecko   (  291): ###!!! [Parent][MessageChannel] Error: Channel error: cannot send/recv
I/Gecko   (  291): 
I/Gecko   (  291): 
I/Gecko   (  291): ###!!! [Parent][MessageChannel] Error: Channel error: cannot send/recv

I'm guessing it's a Gecko regression, since I didn't see this on an earlier Gecko, but I'm going to redirect this to the system app for now. Maybe folks there will have some ideas.

Comment 6

[Jason Smith [:jsmith]]

Switching to qawanted first to do branch checks to confirm comment 5, then move forward with the window.

Comment 7

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Jim Porter (:squib) from comment #5)
> It seems like this happens if you open the settings app, receive a call, and
> then try to change the ringtone. File formats don't seem to matter, and even
> built-in ringtones can break this. I get a lot of this in the logcat when it
> happens:
> 
> I/Gecko   (  291): 
> I/Gecko   (  291): ###!!! [Parent][MessageChannel] Error: Channel error:
> cannot send/recv
> I/Gecko   (  291): 
> I/Gecko   (  291): 
> I/Gecko   (  291): ###!!! [Parent][MessageChannel] Error: Channel error:
> cannot send/recv
> 
> I'm guessing it's a Gecko regression, since I didn't see this on an earlier
> Gecko, but I'm going to redirect this to the system app for now. Maybe folks
> there will have some ideas.

Kyle, could it be one of the settings regression ?

Comment 8

[Adam Alldredge [:AdamA][Inactive]]

I am unable to reproduce this issue and I cannot find the build that you reported it in. Was the initial build a nightly build? or a tinderbox build?

I did see a similar issue that involved "Saving..." being displayed. But it can be reproduced with any ringtone. If the phone times out or is locked on the "Sound" setting screen then the user switches ringtone it will display "Saving..." in the ringtone section and it will not go away until setting are restarted.

the build I was unable to reproduce your issue in but saw the above issue is:

Environmental Variables:
Device: Flame 2.2 Master
BuildID: 20140910060915
Gaia: f108c706fae43cd61628babdd9463e7695b2496e
Gecko: 843332cc69af
Version: 35.0a1 (2.2 Master)
Firmware: V123
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0

Comment 9

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Could be settings, I'm seeing 

E/GeckoConsole(  314): [JavaScript Error: "ReadOnlyError: A mutation operation was attempted in a READ_ONLY transaction." {file: "resource://gre/modules/SettingsRequestManager.jsm" line: 509}]

messages which are fixed by bug 1061510, but which is blocked on uplift by bug 1064228. So once all that is out of the way, this will be worth retesting.

Comment 10

[Jayme Mercado [:JMercado] (Inactive 3/4/2016)]

I have also tried reproducing this issue but could not reproduce it.  Leaving the qawanted tag for others to check again later.

Environmental Variables:
Device: Flame 2.2
BuildID: 20140912061053
Gaia: b72909030e214175144342f7e5df7e88a2b52fd4
Gecko: 59d4326311e0
Version: 35.0a1 (2.2) 
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0

Comment 11

[Candice Serran (:cserran)]

Assigning to you kyle if this is blocked by the settings bug that you linked. Apologies if this isn't correct

Comment 12

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

All dependent bugs landed to aurora today, need confirmation from QA on v2.1 before closing.

Comment 13

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Changing to verifyme since patch already landed.

Comment 14

[Alison Shiue]

Verified on
[master]
Gaia-Rev        3c898380b47f298cd3b7a0dacb3a6529e94322d4
Gecko-Rev       https://hg.mozilla.org/mozilla-central/rev/4cdc4b9e5832
Build-ID        20140922184244
Version         35.0a1
(*note: find another settings crashed issue on master, track on bug 1071431)

[2.1]
Gaia-Rev        3742913e11f69e789dcb0aa0dedf2e5572da0129
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-aurora/rev/df42b05782aa
Build-ID        20140922185144
Version         34.0a2

Comment 15

[Sue]

Attachment: video of verify issue

This issue has been successfully verified on Flame 2.1
See attachment: verify_video.MP4
Reproducing rate: 0/5
Flame 2.1 versions:
Gaia-Rev        1b231b87aad384842dfc79614b2a9ca68a4b4ff3
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/95fbd7635152
Build-ID        20141119001205
Version         34.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141119.035246
FW-Date         Wed Nov 19 03:52:56 EST 2014
Bootloader      L1TC00011880

Comment 16

[Mike Lien[:mlien]]

Hi Kevin, please help to verify if this patch is good on master

Comment 17

[cnelson]

This issue is verified fixed on Flame 2.2.

The set ringtone will save as the default ringtone.

Flame 2.2

Device: Flame 2.2  (319mb)(Kitkat Base)(Shallow Flash)
Build ID: 20141125040209
Gaia: 824a61cccec4c69be9a86ad5cb629a1f61fa142f
Gecko: acde07cb4e4d
Version: 36.0a1 (2.2)
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0