-
-
Notifications
You must be signed in to change notification settings - Fork 443
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
[SR] Fix various crashes #3970
[SR] Fix various crashes #3970
Conversation
…ing GestureRecorder
Performance metrics 🚀
|
Revision | Plain | With Sentry | Diff |
---|---|---|---|
c4a83e5 | 413.86 ms | 464.82 ms | 50.96 ms |
68f8bfc | 414.88 ms | 428.41 ms | 13.53 ms |
113888d | 441.10 ms | 491.92 ms | 50.82 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
c4a83e5 | 1.70 MiB | 2.35 MiB | 669.67 KiB |
68f8bfc | 1.70 MiB | 2.35 MiB | 669.68 KiB |
113888d | 1.70 MiB | 2.35 MiB | 669.67 KiB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Thanks for fixing this! What version of sentry should we update to on Android to incorporate this change? |
@iamrita This fix will ship as part of the |
Great, looking forward to it! |
@markushi Any update on the |
@iamrita sorry for the delay, there was a small hiccup with our release pipeline, but the release is out now! |
📜 Description
IndexOutOfBoundsException
when tracking window changes. The problem here was that we had anobject RootViewSpy
, and it was shared between different instances of the ReplayIntegration, hence messing up things.IllegalStateException
when adding/removing draw listener for a dead viewConcurrentModificationException
when registering window listeners and stoppingWindowRecorder
/GestureRecorder
💡 Motivation and Context
Closes #3859
Closes getsentry/sentry-react-native#4234
💚 How did you test it?
Manually + automated
📝 Checklist
sendDefaultPII
is enabled.🔮 Next steps