918982 - FILE_NOT_FOUND error fired in logcat looking for singleVariant configuration file (singlevariantconf.json)

Status: RESOLVED FIXED

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

Description

[Jason Smith [:jsmith]]

Seeing this in the logcat during daily testing of my Buri 1.2 9/20 build:

09-20 13:16:19.289: E/GeckoConsole(4734): Content JS ERROR at app://homescreen.gaiamobile.org/gaia_build_defer_index.js:365 in loadSVConfFileError: Failed parsing singleVariant configuration file [js/singlevariantconf.json]: [Exception... "File error: Not found"  nsresult: "0x80520012 (NS_ERROR_FILE_NOT_FOUND)"  location: "JS frame :: app://homescreen.gaiamobile.org/gaia_build_defer_index.js :: loadFile :: line 358"  data: no]

We should probably fail a bit more gracefully then just hit a unchecked exception. Probably related to the SIM customization work done.

Comment 1

[Aus Lacroix [:aus]]

Hi Jason, I believe that code was written by the folks at TEF. I'm trying to dig through the commits to see which one contained these changes. I believe this is, however, a blocking issue since it is likely that folks will be shipping builds *without* a customization file for the homescreen apps and we probably don't want to throw exceptions for missing files.

Comment 2

[Albert [:albert]]

Attachment: Patch

Comment 3

[Albert [:albert]]

That message is just to inform, is ok to you changing the console.error by console.log? Or do you want to remove it completely?

Comment 4

[Cristian Rodriguez (:crdlc)]

Comment on attachment 8475849 [details] [review]
Patch

Good job, thanks

Comment 5

[Jason Smith [:jsmith]]

(In reply to Albert [:albert] from comment #3)
> That message is just to inform, is ok to you changing the console.error by
> console.log? Or do you want to remove it completely?

Using console.log works for me.

Comment 6

[Albert [:albert]]

Master https://github.com/mozilla-b2g/gaia/commit/3a7e6277dde31e5a3b9b3861bcf50f105a4669c0