Closed
Bug 1207579
Opened 9 years ago
Closed 7 years ago
crash in java.lang.IllegalAccessException: class org.mozilla.gecko.home.PinSiteDialog is not accessible from class android.support.v4.app.Fragment at java.lang.Class.newInstance(Class.java)
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox55 wontfix, firefox56? fixed, firefox57 fixed)
RESOLVED
FIXED
Firefox 57
People
(Reporter: ioana.chiorean, Assigned: mkaply)
Details
(Keywords: crash)
Crash Data
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
snorp
:
review+
gchang
:
approval-mozilla-beta+
|
Details |
This bug was filed from the Socorro interface and is report bp-3b5a882e-9aed-4820-956f-c60152150923. ============================================================= I am not able to reproduce this with clear steps, but I met it several times in a few hours. Till you kill the app it crashes constantly. I looked in the reports and seems that occurs on several devices.
Reporter | ||
Updated•9 years ago
|
OS: Unspecified → Android
Hardware: Unspecified → ARM
Version: unspecified → Trunk
Reporter | ||
Updated•9 years ago
|
Version: Trunk → Firefox 43
Comment 1•9 years ago
|
||
Tracking for 43. Looks like 41 and 40.0.3 are also affected.
status-firefox40:
--- → affected
status-firefox41:
--- → affected
status-firefox43:
--- → affected
tracking-firefox43:
--- → +
status-firefox42:
--- → ?
Updated•9 years ago
|
Crash Signature: [@ java.lang.IllegalAccessException: class org.mozilla.gecko.home.PinSiteDialog is not accessible from class android.support.v4.app.Fragment at java.lang.Class.newInstance(Class.java)] → [@ java.lang.IllegalAccessException: class org.mozilla.gecko.home.PinSiteDialog is not accessible from class android.support.v4.app.Fragment at java.lang.Class.newInstance(Class.java)]
[@ java.lang.IllegalAccessException: class org.mozilla.gecko.home.Pin…
Comment 2•9 years ago
|
||
Only a couple of crashes on 43 beta; this seems to be a crash that shows up more once we get to the release channel.
Comment 3•8 years ago
|
||
332 crashes on 42 in the last week (most of them startup crashes). Snorp can you help us find an owner for this bug? Or if you think we will end up wontfixing for 43, can you mark it as such? Thanks.
Flags: needinfo?(snorp)
Assignee: nobody → margaret.leibovic
Flags: needinfo?(snorp)
Comment 4•8 years ago
|
||
Currently this only has a couple of crashes on 43 beta 6 and 7. It was a topcrash for 42 though. I'm going to keep tracking this in case it gets worse on release. Maybe it's fixed, maybe it's a different user population.
Comment 5•8 years ago
|
||
199 crashes for 43.0 in the last week still looking like startup crashes. I can't tell how serious that is. Too late to fix in 43 though. This may pop up on 44 once it hits release (in the same pattern).
tracking-firefox44:
--- → ?
I only see 7 instances of this in the past 28 days. I don't think this needs to be tracked for FF44 as it is not a top 20 crasher. If the data changes or there is a spike, please renominate.
status-firefox44:
--- → affected
Updated•8 years ago
|
Assignee: margaret.leibovic → nobody
Comment hidden (mozreview-request) |
Comment 9•7 years ago
|
||
mozreview-review |
Comment on attachment 8893547 [details] Bug 1207579 - Make PinSiteDialog public to ensure it can be accessed from v4.app.fragment https://reviewboard.mozilla.org/r/164622/#review169974 ::: mobile/android/base/java/org/mozilla/gecko/home/PinSiteDialog.java:40 (Diff revision 1) > import android.widget.ListView; > > /** > * Dialog fragment that displays frecency search results, for pinning a site, in a GridView. > */ > class PinSiteDialog extends DialogFragment { The class needs to be public, not the constructor.
Attachment #8893547 -
Flags: review?(snorp) → review-
Comment hidden (mozreview-request) |
Comment 11•7 years ago
|
||
mozreview-review |
Comment on attachment 8893547 [details] Bug 1207579 - Make PinSiteDialog public to ensure it can be accessed from v4.app.fragment https://reviewboard.mozilla.org/r/164622/#review170114
Attachment #8893547 -
Flags: review?(snorp) → review+
Comment 12•7 years ago
|
||
Pushed by mozilla@kaply.com: https://hg.mozilla.org/integration/autoland/rev/044a5b82b056 Make PinSiteDialog public to ensure it can be accessed from v4.app.fragment r=snorp
Assignee | ||
Comment 13•7 years ago
|
||
[Tracking Requested - why for this release]: We had a partner find this in their testing of a preload of Fennec. Since it's such a straightforward fix, I'd like to take it in 56.
tracking-firefox56:
--- → ?
Comment 14•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/044a5b82b056
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Comment 15•7 years ago
|
||
Please nominate it for Beta approval then :)
status-firefox40:
affected → ---
status-firefox41:
affected → ---
status-firefox42:
affected → ---
status-firefox43:
wontfix → ---
status-firefox44:
affected → ---
status-firefox55:
--- → wontfix
status-firefox56:
--- → affected
tracking-firefox43:
+ → ---
tracking-firefox44:
- → ---
Flags: needinfo?(mozilla)
Assignee | ||
Comment 16•7 years ago
|
||
Comment on attachment 8893547 [details] Bug 1207579 - Make PinSiteDialog public to ensure it can be accessed from v4.app.fragment Approval Request Comment [Feature/Bug causing the regression]: N/A [User impact if declined]: Occasional crash [Is this code covered by automated tests?]: No [Has the fix been verified in Nightly?]: Not easily reproducible [Needs manual test from QE? If yes, steps to reproduce]: Not easily reproducible [List of other uplifts needed for the feature/fix]: None [Is the change risky?]: No [Why is the change risky/not risky?]: Change from private to public. No core code change. [String changes made/needed]:
Flags: needinfo?(mozilla)
Attachment #8893547 -
Flags: approval-mozilla-beta?
Comment 17•7 years ago
|
||
Comment on attachment 8893547 [details] Bug 1207579 - Make PinSiteDialog public to ensure it can be accessed from v4.app.fragment A simple change. Beta56+. Should be in 56 beta 2.
Attachment #8893547 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 18•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/316317e8a185
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
•