Closed Bug 796583 Opened 12 years ago Closed 12 years ago

Don't use localStorage in fm.js

Categories

(Firefox OS Graveyard :: Gaia, defect)

defect
Not set
normal

Tracking

(blocking-basecamp:+)

RESOLVED FIXED
blocking-basecamp +

People

(Reporter: ghtobz, Unassigned)

Details

(Whiteboard: [label:perf][label:FM Radio][label:mentored], QARegressExclude)

[GitHub issue by fabricedesre on 2012-08-31T14:28:35Z, https://github.com/mozilla-b2g/gaia/issues/4223]
fm/js/fm.js:396:    var savedList = localStorage.getItem(this.KEYNAME);
fm/js/fm.js:420:    localStorage.setItem(this.KEYNAME, JSON.stringify(this._favList));
[GitHub comment by fabricedesre on 2012-09-18T18:06:56Z]
On 09/18/2012 10:34 AM, Suyash Katiyar wrote:
> Hi @fabricedesre <https://github.com/fabricedesre>,
> Just submitted change for mozilla-b2g/gaia#4222
> <https://github.com/mozilla-b2g/gaia/issues/4222>.
> Would like to work on this issue too.

Sure, please do!
[GitHub comment by ttaubert on 2012-10-01T17:17:35Z]
This has been fixed already, right?
Setting to fixed as all localStorage occurrences were replaced by asyncStorage.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [label:perf][label:FM Radio][label:mentored] → [label:perf][label:FM Radio][label:mentored], QARegressExclude
You need to log in before you can comment on or make changes to this bug.