Closed Bug 1067026 Opened 5 years ago Closed 5 years ago

crash in java.lang.IllegalStateException: Error reading list.txt at org.mozilla.search.providers.SearchEngineManager.getAllEngines(SearchEngineManager.java)

Categories

(Firefox for Android Graveyard :: Search Activity, defect, critical)

35 Branch
All
Android
defect
Not set
critical

Tracking

(firefox35 verified, fennec35+)

VERIFIED FIXED
Firefox 35
Tracking Status
firefox35 --- verified
fennec 35+ ---

People

(Reporter: aaronmt, Assigned: Margaret)

References

Details

(Keywords: crash, reproducible)

Crash Data

Attachments

(1 file)

This bug was filed from the Socorro interface and is 
report bp-c800884e-b259-4058-b8ab-b32882140911.
=============================================================

java.lang.IllegalStateException: Error reading list.txt
	at org.mozilla.search.providers.SearchEngineManager.getAllEngines(SearchEngineManager.java:139)
	at org.mozilla.search.SearchPreferenceActivity$2.doInBackground(SearchPreferenceActivity.java:120)
	at android.os.AsyncTask$2.call(AsyncTask.java:288)
	at java.util.concurrent.FutureTask.run(FutureTask.java:237)
	at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:231)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
	at java.lang.Thread.run(Thread.java:841)

Comments

  "tried opening the settings of the search app"
Growing in volume, tracking?
tracking-fennec: --- → ?
I'll look into this.
Assignee: nobody → margaret.leibovic
This reminds me of the conversation in bug 1065123. This will happen if we can't find the list.txt files, which could happen if we're somehow creating the wrong path for the locale.

Maybe we should land a patch to add some more logging to see what locale we're getting here.

It would be really nice if someone could reproduce this, perhaps by trying out different locales.
Keywords: steps-wanted
My guess: they're using an OS locale that we don't ship.

E.g., es_US. That'll result in a Fennec displayed in English, but the Java locale is still es_US. That'll give us a path of /es-US/, which doesn't exist.

See also Bug 1045053, which is addressing a similar split.
So, STR for you to try:

* Android settings. Language. Pick Español (Estados Unidos).
* Launch Nightly. Verify that it's showing you English, not Spanish.
* Open the search activity and hit settings.
(In reply to Richard Newman [:rnewman] from comment #5)
> So, STR for you to try:
> 
> * Android settings. Language. Pick Español (Estados Unidos).
> * Launch Nightly. Verify that it's showing you English, not Spanish.
> * Open the search activity and hit settings.

Yep that did it.
Blocks: search
How does Fennec deal with this case? Does it just fall back to en-US? We could make the search activity just fall back to en-US instead of crashing.
Flags: needinfo?(rnewman)
If Gecko doesn't understand the OS locale, it believes it's in en-US (which is the source of Bug 1045053).

Falling back to en-US seems like a reasonable choice, yeah :D
Flags: needinfo?(rnewman)
Attachment #8490358 - Flags: review?(rnewman) → review+
https://hg.mozilla.org/mozilla-central/rev/b58ffeb810cd
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 35
tracking-fennec: ? → 35+
There's been 0 crashes reported in the last week but I'm flagging it for QE verification given this has a reproducible case.
Flags: qe-verify+
Verified as fixed in Firefox for Android 35 Beta 1 with Nexus 4 (Android 4.4.4), using steps from comment 5.
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.