Closed
Bug 1507947
Opened 6 years ago
Closed 5 years ago
IllegalArgumentException: Service not registered: org.mozilla.gecko.process.GeckoProcessManager$ChildConnection@f3bf8d0
Categories
(GeckoView :: General, defect, P1)
Tracking
(geckoview64 fixed, firefox64 wontfix, firefox65 fixed, firefox66 fixed)
RESOLVED
FIXED
mozilla66
People
(Reporter: cpeterson, Assigned: snorp)
Details
(Keywords: crash, Whiteboard: [geckoview:fenix:p1])
Crash Data
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
RyanVM
:
approval-mozilla-geckoview64+
|
Details | Review |
There has been one instance of this crash signature from a Focus 8.0 Nightly user with GV 64 Beta. This bug was filed from the Socorro interface and is report bp-c227ee6b-7050-4182-a3b4-6478f0181116. ============================================================= Java stack trace: java.lang.IllegalArgumentException: Service not registered: org.mozilla.gecko.process.GeckoProcessManager$ChildConnection@f3bf8d0 at android.app.LoadedApk.forgetServiceDispatcher(LoadedApk.java:1517) at android.app.ContextImpl.unbindService(ContextImpl.java:1639) at android.content.ContextWrapper.unbindService(ContextWrapper.java:698) at org.mozilla.gecko.process.GeckoProcessManager$ChildConnection.bind(GeckoProcessManager.java:85) at org.mozilla.gecko.process.GeckoProcessManager.start(GeckoProcessManager.java:201) at org.mozilla.gecko.process.GeckoProcessManager.start(GeckoProcessManager.java:189)
Reporter | ||
Comment 1•6 years ago
|
||
P1 because this is GV top crash #3.
Priority: -- → P1
Summary: java.lang.IllegalArgumentException: Service not registered: org.mozilla.gecko.process.GeckoProcessManager$ChildConnection@f3bf8d0 → IllegalArgumentException: Service not registered: org.mozilla.gecko.process.GeckoProcessManager$ChildConnection@f3bf8d0
Reporter | ||
Updated•6 years ago
|
Crash Signature: [@ java.lang.IllegalArgumentException: at android.app.LoadedApk.forgetServiceDispatcher(LoadedApk.java) ] → [@ java.lang.IllegalArgumentException: at android.app.LoadedApk.forgetServiceDispatcher(LoadedApk.java) ]
[@ java.lang.IllegalArgumentException: at android.app.LoadedApk.forgetServiceDispatcher(Unknown Source:195) ]
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → snorp
Reporter | ||
Comment 2•6 years ago
|
||
In anticipation of the creation of the GECKOVIEW_64_RELBRANCH next week, I am moving this bug from status-firefox64=affected to status-geckoview64=affected.
Reporter | ||
Comment 3•5 years ago
|
||
James, do you have any updates on this LoadedApk.forgetServiceDispatcher crash? It's GV top crash #2 for Focus 8.0 beta.
Flags: needinfo?(snorp)
Assignee | ||
Comment 4•5 years ago
|
||
Pushed by jwillcox@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d7b73b7ab1a5 Be more careful when unbinding child process services r=geckoview-reviewers,esawin
Comment 7•5 years ago
|
||
Backed out changeset d7b73b7ab1a5 (Bug 1507947) for mochitest failures on test_expanding_selection_per_page.html. Backout: https://hg.mozilla.org/integration/autoland/rev/a80e4d4824f38ec775a464c05dcf0c853fc2fa26 Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=pending%2Crunning%2Csuccess%2Ctestfailed%2Cbusted%2Cexception&revision=d7b73b7ab1a5edd198b4851eb577331af6e73464&selectedJob=216278924 Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=216278924&repo=autoland&lineNumber=1712
Flags: needinfo?(snorp)
Comment 8•5 years ago
|
||
There are also mochitest crashes failures on layout/base/tests/test_event_target_iframe_oop.html | application crashed [@ mozilla::ipc::GeckoChildProcessHost::OpenPrivilegedHandle(int)]. https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=216276271&repo=autoland&lineNumber=1580
Pushed by jwillcox@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/52c382554814 Be more careful when unbinding child process services r=geckoview-reviewers,esawin
Comment 10•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/52c382554814
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
Assignee | ||
Comment 11•5 years ago
|
||
Comment on attachment 9030235 [details] Bug 1507947 - Be more careful when unbinding child process services [Beta/Release Uplift Approval Request] Feature/Bug causing the regression: N/A User impact if declined: Crashes when using GeckoView Is this code covered by automated tests?: Yes Has the fix been verified in Nightly?: Yes Needs manual test from QE?: No If yes, steps to reproduce: List of other uplifts needed: None Risk to taking this patch: Medium Why is the change risky/not risky? (and alternatives if risky): This change affects how we manage child processes on Android, which is a critical path. It seems to work, though, as we have a bunch of tests that exercise this path. String changes made/needed: None
Flags: needinfo?(snorp)
Attachment #9030235 -
Flags: approval-mozilla-beta?
Assignee | ||
Comment 12•5 years ago
|
||
Comment on attachment 9030235 [details] Bug 1507947 - Be more careful when unbinding child process services [GeckoView Uplift Approval Request] If this is not a sec:{high,crit} bug, please state case for consideration: User impact if declined: Fix Landed on Version: Risk to taking this patch: Low Why is the change risky/not risky? (and alternatives if risky): String or UUID changes made by this patch:
Attachment #9030235 -
Flags: approval-mozilla-geckoview64?
Assignee | ||
Comment 13•5 years ago
|
||
I guess I really wanted approval-mozilla-geckoview64 instead of approval-mozilla-beta
Updated•5 years ago
|
Comment 14•5 years ago
|
||
Comment on attachment 9030235 [details] Bug 1507947 - Be more careful when unbinding child process services [Triage Comment] Fixes a GeckoView crash. Approved for 65.0b5 and GV64.
Attachment #9030235 -
Flags: approval-mozilla-geckoview64?
Attachment #9030235 -
Flags: approval-mozilla-geckoview64+
Attachment #9030235 -
Flags: approval-mozilla-beta?
Attachment #9030235 -
Flags: approval-mozilla-beta+
Comment 15•5 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/d5f982689c71
Comment 16•5 years ago
|
||
uplift |
https://hg.mozilla.org/releases/mozilla-release/rev/448b0b631dd99e6e27559598c1cc191d4cbf10f7
Comment hidden (typo) |
Comment hidden (typo) |
Updated•5 years ago
|
Product: Firefox for Android → GeckoView
Reporter | ||
Updated•5 years ago
|
Whiteboard: [geckoview:fenix:p1]
You need to log in
before you can comment on or make changes to this bug.
Description
•