java.lang.NullPointerException @ Tabs.isSelectedTab(Tabs.java:117)

VERIFIED FIXED

Status

()

Firefox for Android
General
P3
critical
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: aaronmt, Assigned: sriram)

Tracking

({crash, regression, reproducible})

unspecified
ARM
Android
crash, regression, reproducible
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

(Whiteboard: [native-crash])

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
I/GeckoApp( 2871): Got message: Tab:Closed
I/GeckoApp( 2871): Destroyed a tab
I/GeckoTabs( 2871): Removed a tab with id: 2
D/GeckoInputConnection( 2871): IME: run()
D/GeckoInputConnection( 2871): IME: v=org.mozilla.gecko.gfx.LayerView@4058de60
I/GeckoDoorHangerPopup( 2871): Showing all doorhangers for tab: 1
I/GeckoDoorHangerPopup( 2871): Showing all doorhangers for tab: 1
E/GeckoApp( 2871): top level exception
E/GeckoApp( 2871): java.lang.NullPointerException
E/GeckoApp( 2871): 	at org.mozilla.gecko.Tabs.isSelectedTab(Tabs.java:117)
E/GeckoApp( 2871): 	at org.mozilla.gecko.TabsTray$TabsAdapter$2.onClick(TabsTray.java:184)
E/GeckoApp( 2871): 	at android.view.View.performClick(View.java:2485)
E/GeckoApp( 2871): 	at android.view.View$PerformClick.run(View.java:9080)
E/GeckoApp( 2871): 	at android.os.Handler.handleCallback(Handler.java:587)
E/GeckoApp( 2871): 	at android.os.Handler.dispatchMessage(Handler.java:92)
E/GeckoApp( 2871): 	at android.os.Looper.loop(Looper.java:130)
E/GeckoApp( 2871): 	at org.mozilla.gecko.GeckoApp$26.run(GeckoApp.java:1342)
E/GeckoApp( 2871): 	at android.os.Handler.handleCallback(Handler.java:587)
E/GeckoApp( 2871): 	at android.os.Handler.dispatchMessage(Handler.java:92)
E/GeckoApp( 2871): 	at android.os.Looper.loop(Looper.java:130)
E/GeckoApp( 2871): 	at android.app.ActivityThread.main(ActivityThread.java:3683)
E/GeckoApp( 2871): 	at java.lang.reflect.Method.invokeNative(Native Method)
E/GeckoApp( 2871): 	at java.lang.reflect.Method.invoke(Method.java:507)
E/GeckoApp( 2871): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:839)
E/GeckoApp( 2871): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:597)
E/GeckoApp( 2871): 	at dalvik.system.NativeStart.main(Native Method)
D/Zygote  ( 2378): Process 2871 exited cleanly (1)
I/WindowManager( 2385): WIN DEATH: Window{4086ee10 org.mozilla.fennec/org.mozilla.fennec.App paused=false}
I/ActivityManager( 2385): Process org.mozilla.fennec (pid 2871) has died.
I/WindowManager( 2385): WIN DEATH: Window{40888118 SurfaceView paused=false}

STR:

1. http://espn.com, http://m.engadget.com
2. Switch between tabs, and then close the first tab

--
Samsung Galaxy SII (Android 2.3.4)
20111205040214
http://hg.mozilla.org/projects/birch/rev/85e665cc4a97
(Reporter)

Updated

6 years ago
Assignee: nobody → sriram
(Assignee)

Comment 1

6 years ago
Created attachment 579469 [details] [diff] [review]
Patch

Unfortunately, I couldn't reproduce the bug as always.
However, I've added a preventive check to avoid an NPE.
Also, I've cleaned up some code.
Attachment #579469 - Flags: review?(mark.finkle)
(Assignee)

Updated

6 years ago
Depends on: 701884
Priority: -- → P3
Attachment #579469 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/bb4d6dffb53d

Comment 3

6 years ago
https://hg.mozilla.org/mozilla-central/rev/bb4d6dffb53d
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
tracking-fennec: --- → 11+
status-firefox11: --- → fixed
This crash cannot be reproduced anymore on the latest Nightly build. Closing bug as verified fixed. Please file a new bug if this crash will occur again.

--
Mozilla/5.0 (Android;Linux armv7l;rv:12.0a1)Gecko/2012012
Firefox/12.0a1 Fennec/12.0a1
Device: Samsung Nexus S
OS: Android 2.3.6
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.