Closed
Bug 1270522
Opened 8 years ago
Closed 8 years ago
Crash in java.lang.NullPointerException: at org.mozilla.gecko.gfx.GeckoLayerClient.sendResizeEventIfNecessary(GeckoLayerClient.java)
Categories
(Firefox for Android Graveyard :: Toolbar, defect)
Tracking
(firefox47+ fixed, firefox48 fixed, firefox49 fixed, fennec47+)
RESOLVED
FIXED
Firefox 49
People
(Reporter: kbrosnan, Assigned: snorp)
Details
(Keywords: crash, regression, topcrash-android-armv7)
Crash Data
Attachments
(1 file)
1.46 KB,
patch
|
rbarker
:
review+
ritu
:
approval-mozilla-aurora+
ritu
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
This bug was filed from the Socorro interface and is report bp-b6d19e05-734e-4b9d-bd7f-ea7c92160504. ============================================================= java.lang.NullPointerException at org.mozilla.gecko.gfx.GeckoLayerClient.sendResizeEventIfNecessary(GeckoLayerClient.java:284) at org.mozilla.gecko.gfx.GeckoLayerClient.onGeckoReady(GeckoLayerClient.java:164) at org.mozilla.gecko.mozglue.GeckoLoader.nativeRun(Native Method) at org.mozilla.gecko.GeckoThread.run(GeckoThread.java:443)
Reporter | ||
Updated•8 years ago
|
Assignee: nobody → snorp
tracking-fennec: ? → 47+
James, is this something that worries you? Let me know if you don't have time for it, or unassign, and I'll find somebody.
Flags: needinfo?(snorp)
This is currently the #8 crash in Fennec Beta accounting for 1.64% of the overall volume. It's also categorized as a startup crash since 63% of the reports happen within 1 minute of uptime. Strangely it appears as though these reports are isolated to the 47 branch which makes me think this issue could be under a different signature in other branches.
Keywords: topcrash-android-armv7
Assignee | ||
Comment 3•8 years ago
|
||
Attachment #8755437 -
Flags: review?(rbarker)
Updated•8 years ago
|
Attachment #8755437 -
Flags: review?(rbarker) → review+
This looks like a good candidate for an uplift, nullptr check, and this regressed in 47, so we could catch up and not ship it at all.
Version: Trunk → 47 Branch
Updated•8 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 7•8 years ago
|
||
Comment on attachment 8755437 [details] [diff] [review] Guard against a null view in GeckoLayerClient.sendResizeEventIfNecessary() Approval Request Comment [Feature/regressing bug #]: Unknown [User impact if declined]: Crashes [Describe test coverage new/current, TreeHerder]: nightly [Risks and why]: low, only adds a null check [String/UUID change made/needed]: none
Attachment #8755437 -
Flags: approval-mozilla-beta?
Comment on attachment 8755437 [details] [diff] [review] Guard against a null view in GeckoLayerClient.sendResizeEventIfNecessary() Crash fix, there's ~200 crashes in a week on 47.0b6 and the fix is a simple not-null check, Aurora48+, Beta47+
Attachment #8755437 -
Flags: approval-mozilla-beta?
Attachment #8755437 -
Flags: approval-mozilla-beta+
Attachment #8755437 -
Flags: approval-mozilla-aurora+
Comment 9•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5abd0e13a583
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Comment 10•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/94069e0bf766
Comment 11•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/5564b762402e
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•