Closed Bug 1129840 Opened 10 years ago Closed 8 years ago

Tabs are not always restored on Gingerbread devices


(Firefox for Android Graveyard :: General, defect)

Not set



Tracking Status
fennec + ---


(Reporter: cos_flaviu, Unassigned)


Device: Samsung Galaxy R (Android 2.3.4);
Build: Nightly 38.0a1 (2015-02-04);

Steps to reproduce:
1. Go to Settings -> Customize -> Tabs and enable Always restore tabs;
2. Load a few pages each in new tab;
3. Open a new tab and go to about:home;
4. While in about:home tap on Options -> Quit.
5. Launch fennec.

Expected result:
Fennec is launched and all the opened tabs from step 2 are restored.

Actual result:
None of the opened tabs are restored; Fennec launches with one about:home tab.

Not reproducible every time. 
The bug is specific to Gingerbread devices because only those have the Quit option in menu.

The following exceptions are thrown when launching Fennec.

E/GeckoSessionParser(3329): JSON error
E/GeckoSessionParser(3329): org.json.JSONException: Index 0 out of range [0..0)
E/GeckoSessionParser(3329): 	at org.json.JSONArray.get(
E/GeckoSessionParser(3329): 	at org.json.JSONArray.getJSONObject(
E/GeckoSessionParser(3329): 	at org.mozilla.gecko.SessionParser.parse(
E/GeckoSessionParser(3329): 	at org.mozilla.gecko.GeckoApp.restoreSessionTabs(
E/GeckoSessionParser(3329): 	at org.mozilla.gecko.GeckoApp.onWindowFocusChanged(
E/GeckoSessionParser(3329): 	at org.mozilla.gecko.BrowserApp.onWindowFocusChanged(
E/GeckoSessionParser(3329): 	at$DecorView.onWindowFocusChanged(
E/GeckoSessionParser(3329): 	at android.view.View.dispatchWindowFocusChanged(
E/GeckoSessionParser(3329): 	at android.view.ViewGroup.dispatchWindowFocusChanged(
E/GeckoSessionParser(3329): 	at android.view.ViewRoot.handleMessage(
E/GeckoSessionParser(3329): 	at android.os.Handler.dispatchMessage(
E/GeckoSessionParser(3329): 	at android.os.Looper.loop(
E/GeckoSessionParser(3329): 	at
E/GeckoSessionParser(3329): 	at java.lang.reflect.Method.invokeNative(Native Method)
E/GeckoSessionParser(3329): 	at java.lang.reflect.Method.invoke(
E/GeckoSessionParser(3329): 	at$
E/GeckoSessionParser(3329): 	at
E/GeckoSessionParser(3329): 	at dalvik.system.NativeStart.main(Native Method)
E/GeckoApp(3329): An error occurred during restore
E/GeckoApp(3329): org.mozilla.gecko.GeckoApp$SessionRestoreException: No tabs could be read from session file
E/GeckoApp(3329): 	at org.mozilla.gecko.GeckoApp.restoreSessionTabs(
E/GeckoApp(3329): 	at org.mozilla.gecko.GeckoApp.onWindowFocusChanged(
E/GeckoApp(3329): 	at org.mozilla.gecko.BrowserApp.onWindowFocusChanged(
E/GeckoApp(3329): 	at$DecorView.onWindowFocusChanged(
E/GeckoApp(3329): 	at android.view.View.dispatchWindowFocusChanged(
E/GeckoApp(3329): 	at android.view.ViewGroup.dispatchWindowFocusChanged(
E/GeckoApp(3329): 	at android.view.ViewRoot.handleMessage(
E/GeckoApp(3329): 	at android.os.Handler.dispatchMessage(
E/GeckoApp(3329): 	at android.os.Looper.loop(
E/GeckoApp(3329): 	at
E/GeckoApp(3329): 	at java.lang.reflect.Method.invokeNative(Native Method)
E/GeckoApp(3329): 	at java.lang.reflect.Method.invoke(
E/GeckoApp(3329): 	at$
E/GeckoApp(3329): 	at
E/GeckoApp(3329): 	at dalvik.system.NativeStart.main(Native Method)
tracking-fennec: --- → ?
Is this a recent regression?
Is 37 affected? Regression from bug 1015516?
This could be related to the formdata data patch I landed in bug 671993.
Assignee: nobody → mark.finkle
tracking-fennec: ? → 38+
I'm getting this in my local build on my N7 (non-GB) by swiping-to-close and reinstalling a new build (which kills the old one)
Any news, mfinkle? Should this be reassigned?
Flags: needinfo?(mark.finkle)
Assignee: mark.finkle → nobody
tracking-fennec: 38+ → +
Flags: needinfo?(mark.finkle)
We no longer support Gingerbread, although by enabling the data clearing options or through an add-on the Quit menu can still be used. In any case I think that with the recent work on the session store (in this context especially bug 1228593) this should work now.
Closed: 8 years ago
Resolution: --- → WORKSFORME
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.