Closed Bug 820022 Opened 10 years ago Closed 10 years ago

java.lang.NullPointerException: at org.mozilla.gecko.GeckoApp.onBackPressed(


(Firefox for Android Graveyard :: General, defect)

20 Branch
Not set


(firefox20- wontfix, firefox21+ verified, firefox22 verified, firefox23 fixed)

Firefox 23
Tracking Status
firefox20 - wontfix
firefox21 + verified
firefox22 --- verified
firefox23 --- fixed


(Reporter: scoobidiver, Assigned: wesj)


(Keywords: crash, regression, topcrash, Whiteboard: [native-crash])

Crash Data


(1 file)

Bug 781143 is not fully fixed.
Indeed, there's one crash in 20.0a1/20121210: bp-9d8a9715-faac-4060-be4a-ddac92121210.

	at org.mozilla.gecko.GeckoApp.onBackPressed(
	at android.view.KeyEvent.dispatch(
	at android.view.ViewRootImpl.deliverKeyEventPostIme(
	at android.view.ViewRootImpl.deliverKeyEvent(
	at android.view.ViewRootImpl.deliverInputEvent(
	at android.view.ViewRootImpl.doProcessInputEvents(
	at android.view.ViewRootImpl.enqueueInputEvent(
	at android.view.ViewRootImpl$WindowInputEventReceiver.onInputEvent(
	at android.view.InputEventReceiver.dispatchInputEvent(
	at android.os.MessageQueue.nativePollOnce(Native Method)
	at android.os.Looper.loop(
	at java.lang.reflect.Method.invokeNative(Native Method)
	at java.lang.reflect.Method.invoke(
	at dalvik.system.NativeStart.main(Native Method)

More reports at:
Assignee: nobody → margaret.leibovic
I don't have the cycles to work on this right now.
Assignee: margaret.leibovic → nobody
It's #14 top crasher in 20.0a1.
Keywords: regression
Version: Trunk → Firefox 20
It's #9 top crasher in 20.0a2 and #54 in 21.0a1.
Keywords: topcrash
The crash volume is so low that we're going to leave this on the nom list until after b1 of FF20 is released. Until that time, not worth devoting engineering attention.
Since we released the mobile beta on Monday we're still waiting here for more data collection.
Volume appears stable here, at #15 for Fennec top crashes, not tracking.
Keywords: topcrash
Hey Aaronmt, do you have a moment to try taking a stab at figuring out where the null pointer check might need to go?
Flags: needinfo?(aaron.train)
Keywords: qawanted
Better to go with Wes who reviewed Margaret's patch the first time-around; just about everything in onBackPressed already looks like it has a null-check.
Flags: needinfo?(wjohnston)
Flags: needinfo?(aaron.train)
Flags: needinfo?
Attached patch Patch?Splinter Review
This is the only thing that stood out to me. mLayerView is initialized in initializeChrome as well and might be null? Thant matches with the line number from here:
Attachment #730754 - Flags: review?(margaret.leibovic)
Flags: needinfo?(wjohnston)
Flags: needinfo?
Attachment #730754 - Flags: review?(margaret.leibovic) → review+
Request uplift please. We want this on Fx22 at a minimum.
Assignee: nobody → wjohnston
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
It's #8 top crasher in 20.0.
tracking-fennec: --- → ?
Keywords: topcrash
(In reply to Wesley Johnston (:wesj) from comment #10)

Wes, can you please request nomination for uplift here ?
Comment on attachment 730754 [details] [diff] [review]

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Forever
User impact if declined: Crashes if you hit back really early in startup
Testing completed (on m-c, etc.): Landed on mc awhile ago. Looks good!
Risk to taking this patch (and alternatives if risky): Very low risk. Just a null check
String or IDL/UUID changes made by this patch: None.
Attachment #730754 - Flags: approval-mozilla-beta?
Attachment #730754 - Flags: approval-mozilla-aurora?
Comment on attachment 730754 [details] [diff] [review]

low risk, just a null check and avoids a top-crash. Approving on beta/aurora.
Attachment #730754 - Flags: approval-mozilla-beta?
Attachment #730754 - Flags: approval-mozilla-beta+
Attachment #730754 - Flags: approval-mozilla-aurora?
Attachment #730754 - Flags: approval-mozilla-aurora+
There are no crashes in 21.0b2 and above and after 22.0a2/20130408.
Keywords: qawanted
tracking-fennec: ? → ---
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.