-
Notifications
You must be signed in to change notification settings - Fork 694
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
App freezes, DeadObjectException #925
Comments
More details on the phone that experiences the issue:
|
Thanks for the info will look into it. @hannta the reason why it does not happen on 6.0.1 is we were not on 5.2.+ of the Mapbox Android SDK that made changes to it's location provider deps |
This is bug blocking our next release :/ |
Is this happening on more than just the Sony XPERIA? |
@nitaliano actually it looks like it happens only on this Sony XPERIA device, but we have very limited test device set atmo. |
I was able to get my hands on an Xperia running 5.1.1, I'll see if I can reproduce the issue |
Great news! |
I can't reproduce this issue with my Xperia. Are there any specific steps I need to take in order to get this error? |
We also tested on Android Huawei ALE-L21 and we got it freeze like with Sony. |
Probably worth mentioning that the issue gets reproduced on XPERIA phone in question every time while on Huawei it happens every 2nd-3rd time. It looks like the root cause is some raise condition that takes place more often on a slower phone. |
I am doing repro repo so you can try it also @nitaliano
|
I got this freeze reproduced in small repo! 🎉 It is here: https://github.com/henrikra/mapboxFreeezeBug Steps to repro the freeze:
If you look closely to the video I get crash on 3rd time. And on 3rd time when the modal starts to animate from bottom it does not have black background. It map initializing when there is quick black background? Does this help you? I recorded the video on: |
@henrikra thanks I will check it out with my Xperia |
@henrikra I needed to request access to see the video, let me know when you grant it |
I can reproduce it now, not the DeadObjectException but the error that @henrikra posted Update: I can reproduce both errors |
Commenting out Lost stops the DeadObjectException from happening, but the NavigationActivity from react-native-navigation is still creating the ANR. It looks like it happens when the NavigationActivity is not completely disposed of by the time we push a new NavigationActivity Update: |
@nitaliano Hi! I'm having a similar issue. The application crashes with DeadObjectException:
I'm able to consistently trigger this by drawing a large image to map with ImageSource + RasterLayer, and then zooming in to the image as close as possible, and then repeatedly slightly zooming out/in (hold one finger still and move one back and forth). This results in a significant performance drop followed by a crash, but without any "Unfortunately App has stopped" popups or ANRs. Even though this log is from Android 4.3, I'm able to reproduce this on all my devices:
Still, it looks to be easier to reproduce on lower performance devices. On Nexus 5X it takes around 20s of fiddling to crash, and around 6s on S3. Once you notice a hitch in framerate, logs show all kinds of problems. If you stop fiddling there, the app won't necessarily crash (Log from Android 8.1.0: https://pastebin.com/ztvgbXFB). This suggests that it's not really DeadObjectException crashing the app (at least for me), but rather the exception is just another symptom. Here's a log from fiddling until a crash on Android 8.1.0: https://pastebin.com/weve75PB I tried adding I'm using Mapbox Android SDK 5.3.1 |
Actually I think we can try two things to learn from information about this
|
I was using a custom location engine, so I don't think that makes a difference. That being said, I'll try to create a minimal app on Monday that I can reproduce the issue with, similar to @henrikra's repo. |
there is a fix out for this issue #1001 |
We tried both #1001 and #1010 and those PR's don't help with this problem. Then we went back to vanilla But when using Btw what the TLDR
Logs when map is black:
|
I had some trouble recreating the crash from scratch, and I was only able to reproduce the crash with a custom style. It's likely I'm having a separate issue. I'll open a separate issue somewhere once I have the time to complete an app that can reproduce the crash I'm experiencing. |
Hi @nitaliano, is there any updates on this? |
@nitaliano Can you tell some status of this? This is number one priority for us! :/ |
I'm focusing on this issue this week, I want to fix this before the final 6.0.3 release |
When I comment out the UPDATE: @henrikra to answer your question above |
Can someone test this PR out #1033. I don't have the Sony phone on me but I'm no longer running into the ANR's that were being created before. |
Huawei P9 Plus
|
@ricardochen I would recommend using a I can help you out if you need to make the switch to sources/layers we have a gitter https://gitter.im/react-native-mapbox-gl/Lobby?source=orgpage where you can ask me or anyone in the community anything you want |
@nitaliano We are using also |
@henrikra yes, this shouldn't be an issue now with |
Hello @henrikra I have app freeze issue with How to reproduce crash:
|
Experiencing the above exception on a Huawei P9 Light using the latest dependencies:
|
Any update? |
iOS release and debug works like a charm, android debug works but release crashes because of this. Samsung Xcover 3, android 5.1.1 |
On android im getting following app freezes, DeadObjectException, on some older Android devices.
I think this is similar issue, already reported on mapbox android sdk mapbox/mapbox-navigation-android#522
At least in my case, this issue is reproducing only on react-native-mapbox-gl: 6.0.2, not 6.01.
react-native-mapbox-gl: 6.0.2
react-native: 0.51.0
device: android
The text was updated successfully, but these errors were encountered: