Closed
Bug 1095190
Opened 10 years ago
Closed 10 years ago
"Ringtone not created" error when trying to set a Ringtone from a Music track
Categories
(Firefox OS Graveyard :: Gaia::Ringtones, defect)
Tracking
(b2g-v2.2 affected)
RESOLVED
FIXED
2.2 S4 (23jan)
Tracking | Status | |
---|---|---|
b2g-v2.2 | --- | affected |
People
(Reporter: marcia, Assigned: squib)
Details
(Whiteboard: [2.2-flame-reduced-run])
Attachments
(6 files)
Flame, while running:
Gaia 068b9711277b06c7d633517f9e1fcb5624bb39b3
SourceStamp a074c0112919
BuildID 20141106040207
Version 36.0a1
v 188
STR:
1. Open the Music app and select a ringtone.
2. Set the ringtone as the default
3. Go back and select a different track
4. Received the attached error
Logcat attached.
Reporter | ||
Comment 1•10 years ago
|
||
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-v2.2:
--- → affected
Flags: needinfo?(dharris)
Whiteboard: [2.2-flame-reduced-run]
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(dharris)
This issue could be repro on lasted build of Flame 2.2
STR:
1. Launch Music app.
2. Select a music to play.
3. Tap share button -> Ringtones.
4. Tap Save button.
**It will remind "Ringtone Not Created".
Found time:15:37
See attachment: logcat_1537.txt
Flame 2.2 build:
Gaia-Rev 3554ea9504046646b4679e3a61317c49fc55ca87
Gecko-Rev https://hg.mozilla.org/mozilla-central/rev/67c42c076393
Build-ID 20141228010205
Version 37.0a1
Device-Name flame
FW-Release 4.4.2
FW-Incremental eng.cltbld.20141228.045435
FW-Date Sun Dec 28 04:54:46 EST 2014
Bootloader L1TC00011880
Assignee | ||
Comment 3•10 years ago
|
||
You're hitting a quote in indexedDB:
12-29 15:37:39.143 W/Ringtones( 6899): [JavaScript Error: "QuotaExceededError" {file: "app://ringtones.gaiamobile.org/gaia_build_defer_share.js" line: 323}]
Probably fallout from bug 1083927, based on the dates.
Flags: needinfo?(bent.mozilla)
Assignee | ||
Comment 4•10 years ago
|
||
(In reply to Jim Porter (:squib) from comment #3)
> You're hitting a quote in indexedDB:
Oops, I mean "quota".
Do all the apps involved have the correct storage permissions already? If not then we should add them.
Otherwise, maybe Jan has an idea?
Flags: needinfo?(bent.mozilla) → needinfo?(Jan.Varga)
Assignee | ||
Comment 6•10 years ago
|
||
Here are the permissions for the ringtones app, which should be the only relevant app in this situation: https://github.com/mozilla-b2g/gaia/blob/master/apps/ringtones/manifest.webapp#L11-L17
Were new permissions added for storing large blobs in IndexedDB? Or should we migrate away from storing them in there?
I think the ringtone app probably needs the "storage" permission also. Try it at least.
Fabrice, thoughts?
Flags: needinfo?(fabrice)
Updated•10 years ago
|
Flags: needinfo?(Jan.Varga)
Comment 8•10 years ago
|
||
(In reply to ben turner [:bent] (use the needinfo? flag!) from comment #7)
> I think the ringtone app probably needs the "storage" permission also. Try
> it at least.
>
> Fabrice, thoughts?
Yes, it should ask "storage" to get unlimited indexedDB storage (since we don't implement prompting).
Flags: needinfo?(fabrice)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][COM=Gaia::Ringtones]
Thanks fabrice.
Jim, I think adding the 'storage' permission to your manifest will fix the issue.
Flags: needinfo?(squibblyflabbetydoo)
Comment 10•10 years ago
|
||
Right, add the permission.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → squibblyflabbetydoo
Status: NEW → ASSIGNED
Flags: needinfo?(squibblyflabbetydoo)
Assignee | ||
Comment 11•10 years ago
|
||
This fixes things.
Assignee | ||
Updated•10 years ago
|
Attachment #8546146 -
Flags: review?(hub)
Comment 12•10 years ago
|
||
Comment on attachment 8546146 [details] [review]
Fix it
r=me
Attachment #8546146 -
Flags: review?(hub) → review+
Assignee | ||
Comment 13•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S4 (23jan)
Comment 14•10 years ago
|
||
This issue has been verified successfully on Flame 2.2
STR:
1. Launch Music app.
2. Select a music to play.
3. Tap share button -> Ringtones.
4. Tap Save button.
**Music can be save as Ringtones.
Flame 2.2 build:
Gaia-Rev f5e481d4caf9ffa561720a6fc9cf521a28bd8439
Gecko-Rev https://hg.mozilla.org/mozilla-central/rev/bb8d6034f5f2
Build-ID 20150111010223
Version 37.0a1
Device-Name flame
FW-Release 4.4.2
FW-Incremental eng.cltbld.20150111.043244
FW-Date Sun Jan 11 04:32:55 EST 2015
Bootloader L1TC000118D0
Status: RESOLVED → VERIFIED
Comment 15•10 years ago
|
||
I have verified it again after restarting device, and I found the verification failed on Flame 2.2. I don't know the reason why it is verified successfully in the first time.
Found time: 18:06
See attachment:logcat_1806.txt and 1806.MP4
STR:
1. Launch Music app.
2. Select a music to play.
3. Tap share button -> Ringtones.
4. Tap Save button.
**It will remind "Ringtone Not Created".
Flame 2.2 build:
Gaia-Rev f5e481d4caf9ffa561720a6fc9cf521a28bd8439
Gecko-Rev https://hg.mozilla.org/mozilla-central/rev/bb8d6034f5f2
Build-ID 20150111010223
Version 37.0a1
Device-Name flame
FW-Release 4.4.2
FW-Incremental eng.cltbld.20150111.043244
FW-Date Sun Jan 11 04:32:55 EST 2015
Bootloader L1TC000118D0
Assignee | ||
Comment 16•10 years ago
|
||
It's still a quota issue. Make sure you've flashed a completely fresh build; this includes wiping your old profile. If you don't do that, it won't work.
Comment 17•10 years ago
|
||
1. After flash build, launch Music app and set a music as ringtones successfully.
2. Restart device, and verified it again, it will remind "Ringtone Not Created".
Assignee | ||
Comment 18•10 years ago
|
||
I'm unable to reproduce this issue after the fix landed. Could QA take a look and see if they can reproduce?
Keywords: qawanted
Comment 19•10 years ago
|
||
I was able to reproduce this issue on an earlier build (BuildID: 2015011201022) on Flame 2.2. However, after trying about 20 times, I was unable to reproduce the issue on Flame Master and 2.2.
Result: Ringtones are created as default without any error messages.
Device: Flame Master
Build ID: 20150224010314
Gaia: 31ac1cd7a029d5e46dd7c92537b5c973c5d9826e
Gecko: 368c62292249
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 39.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0
Device: Flame 2.2 (KK, 319mb, full flash)
Build ID: 20150224002637
Gaia: 8e98fe665f3821d10d4d982cbb14cbe5b94d0be5
Gecko: 2b70d9d62799
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 37.0 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
QA Whiteboard: [QAnalyst-Triage+][COM=Gaia::Ringtones] → [QAnalyst-Triage?][COM=Gaia::Ringtones]
Flags: needinfo?(ktucker)
Keywords: qawanted
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?][COM=Gaia::Ringtones] → [QAnalyst-Triage+][COM=Gaia::Ringtones]
Flags: needinfo?(ktucker)
Assignee | ||
Comment 20•10 years ago
|
||
From comment 19, it sounds like this is indeed fixed. Closing.
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•