Closed Bug 1267675 Opened 8 years ago Closed 8 years ago

gaia shared IconsHelper doesn't handle lack of datastore

Categories

(Firefox OS Graveyard :: Gaia, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: cwiiis, Assigned: cwiiis)

References

Details

Attachments

(1 file)

If you try to fetch an icon using IconsHelper, it will fail because datastore is no longer available. There are comments in the code that say it should still fetch the icon, but it doesn't.

Of course we should get it to stop using datastore too, but we should also fix the error handling. This bug is about the latter.
Comment on attachment 8745385 [details] [review]
[gaia] Cwiiis:fix-iconshelper-error-handling > mozilla-b2g:kanikani

There may be other functions that are still broken, I didn't audit the file - this is enough to make homescreen work.
Attachment #8745385 - Flags: review?(apastor)
Comment on attachment 8745385 [details] [review]
[gaia] Cwiiis:fix-iconshelper-error-handling > mozilla-b2g:kanikani

LGTM. Thanks!
Attachment #8745385 - Flags: review?(apastor) → review+
Merged: https://github.com/mozilla-b2g/gaia/commit/0b0d254562f384ad401cf240b8e6ab58d3244623
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: