1149293 - [Customizer] Crash in FM Radio when saving a new add-on

Status: RESOLVED INVALID

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

Description

[Justin D'Arcangelo [:justindarc]]

STR:

1. Flash latest fxos/lightsaber (DEVICE_DEBUG=1 NOFTU=1 GAIA_DEV_PIXELS_PER_PX=2.25 make reset-gaia)
2. Settings -> Add-ons -> Customizer -> Enable
3. Open FM Radio
4. Open Customizer (two-finger swipe up from bottom)
5. Select an element in the DOM tree
6. Long-press the element and select "Remove"
7. Tap "OK" to save the add-on
8. **CRASH**

Comment 1

[[:fabrice] Fabrice Desré]

Testing on a flame, I can't reproduce the crash. However I get sdcard errors in logcat:

I/Gecko   (21972): ##### DeviceStorage: GetVolumeByName('sdcard1') failed

or if using internal storage as default:

I/Gecko   (21972): ##### DeviceStorage: GetVolumeByName('sdcard') failed

Not sure what's happening since the media apps get images/video/music files as expected.

Comment 2

[Justin D'Arcangelo [:justindarc]]

(In reply to Fabrice Desré [:fabrice] from comment #1)
> Testing on a flame, I can't reproduce the crash. However I get sdcard errors
> in logcat:
> 
> I/Gecko   (21972): ##### DeviceStorage: GetVolumeByName('sdcard1') failed
> 
> or if using internal storage as default:
> 
> I/Gecko   (21972): ##### DeviceStorage: GetVolumeByName('sdcard') failed
> 
> Not sure what's happening since the media apps get images/video/music files
> as expected.

I spoke to dhylands about this a few weeks ago and its a false alarm. I think its something to do with the DeviceStorage logs you're seeing are coming from a different process.

Comment 3

[[:fabrice] Fabrice Desré]

ok, but then I see no crash but no new add-on created either.

Comment 4

[Justin D'Arcangelo [:justindarc]]

Bulk update to set the correct component and whiteboard.

Comment 5

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

not a spark blocker as FM Radio is removed for the spark build currently.

Comment 6

[Justin D'Arcangelo [:justindarc]]

Closing this as invalid since FM Radio is no longer part of the build.