Crash in [@ java.lang.IllegalStateException: at android.app.ContextImpl.startServiceCommon(ContextImpl.java)]
Categories
(Firefox for Android Graveyard :: General, defect, P1)
Tracking
(firefox65 wontfix, firefox66 wontfix, firefox67 fixed)
People
(Reporter: calixte, Assigned: vlad.baicu)
Details
(Keywords: crash)
Crash Data
Attachments
(2 files)
This bug is for crash report bp-136fb172-9ad8-4401-ada4-358ce0190227.
Java stack trace:
java.lang.IllegalStateException
at android.app.ContextImpl.startServiceCommon(ContextImpl.java:1595)
at android.app.ContextImpl.startService(ContextImpl.java:1550)
at android.content.ContextWrapper.startService(ContextWrapper.java:664)
at org.mozilla.gecko.LauncherActivity.dispatchTabQueueIntent(LauncherActivity.java:107)
at org.mozilla.gecko.LauncherActivity.onCreate(LauncherActivity.java:84)
at android.app.Activity.performCreate(Activity.java:7149)
at android.app.Activity.performCreate(Activity.java:7140)
at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1288)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3017)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3172)
at android.app.servertransaction.LaunchActivityItem.execute(LaunchActivityItem.java:78)
at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:108)
at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:68)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1906)
at android.os.Handler.dispatchMessage(Handler.java:106)
at android.os.Looper.loop(Looper.java:193)
at android.app.ActivityThread.main(ActivityThread.java:6863)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:537)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:858)
This crash is here for a moment (even after the backout in bug 1510587).
The # of crashes in release 65 is greater than 3.500.
:droeh, could you investigate please ?
Comment 1•3 years ago
|
||
full java stack from one of the other crash ids
java.lang.IllegalStateException: Not allowed to start service Intent { act=android.intent.action.VIEW dat=http://connectivitycheck.platform.hicloud.com/... flg=0x10c00000 hwFlg=0x100 cmp=org.mozilla.firefox/org.mozilla.gecko.tabqueue.TabQueueService }: app is in background uid UidRecord{cf6d35c u0a136 TPSL idle procs:1 seq(0,0,0)} at
android.app.ContextImpl.startServiceCommon(ContextImpl.java:1806) at
android.app.ContextImpl.startService(ContextImpl.java:1761) at
android.content.ContextWrapper.startService(ContextWrapper.java:664) at
org.mozilla.gecko.LauncherActivity.dispatchTabQueueIntent(LauncherActivity.java:107) at
org.mozilla.gecko.LauncherActivity.onCreate(LauncherActivity.java:84) at
android.app.Activity.performCreate(Activity.java:7455) at
android.app.Activity.performCreate(Activity.java:7445) at
android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1286) at
android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3343) at
android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3548) at
android.app.servertransaction.LaunchActivityItem.execute(LaunchActivityItem.java:86) at
android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:108) at
android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:68) at
android.app.ActivityThread$H.handleMessage(ActivityThread.java:2155) at
android.os.Handler.dispatchMessage(Handler.java:109) at
android.os.Looper.loop(Looper.java:207) at
android.app.ActivityThread.main(ActivityThread.java:7534) at
java.lang.reflect.Method.invoke(Native Method) at
com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:524) at
com.android.internal.os.ZygoteInit.main(ZygoteInit.java:958)
Updated•3 years ago
|
Comment 2•3 years ago
|
||
This isn't a GeckoView issue. The problem (as with a few other bugs at this point) is most likely the new service restrictions in Android 8. Bouncing the NI to Stefan.
This is a fairly high volume crash on release. I'll follow up in email.
Marcia also mentioned this is showing up more strongly for Android 9 than in 8.
Comment 5•3 years ago
|
||
Indeed, this is clearly because of the new service restrictions in Android 8.
IllegalStateException: Not allowed to start service: app is in background
Since the tab queue functionality already means a system notification about it for the user, we can convert this service to a foreground service which can be started even with the app in background.
Assignee | ||
Updated•3 years ago
|
Comment 6•3 years ago
|
||
Can you explain how exactly this would work, Vlad? I want to understand what will be shown exactly, and what the buttons do.
Assignee | ||
Comment 7•3 years ago
|
||
Currently if you have the tab queue enabled, trying to open a link will queue it in our app and display a view on top of the apps you are currently using as well as show a notification with how many tabs are currently queued.
By refactoring this service to a foreground one, we need to start it with a notification asap and lock it for the duration of the service's life. My plan is to use the same notification but the user will be unable to swipe it away. Instead I suggest we add a "close" button in order to dismiss the feature.
Assignee | ||
Comment 8•3 years ago
|
||
Assignee | ||
Comment 9•3 years ago
|
||
Refactored the TabQueueService to be a foreground service from Android O
onwards. The service now uses a foreground notification that briefly informs
the user that a new tab is being added to the queue.
Depends on D23528
Assignee | ||
Updated•3 years ago
|
Comment 10•3 years ago
|
||
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/autoland/rev/069dea6b6bc5
Part 1: Some code cleanup. r=petru
https://hg.mozilla.org/integration/autoland/rev/36132f5347e8
Part 2: Refactor TabQueue to a foreground service. r=JanH
Comment 11•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/069dea6b6bc5
https://hg.mozilla.org/mozilla-central/rev/36132f5347e8
Updated•2 years ago
|
Description
•