-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Add new LeakCanary Plumber library #1295
Comments
cc @bishiboosh who mentioned that idea |
Some leaks have clear triggers on when to try and fix them, while others less so and we've typically used a timer based approach in the past. I'm thinking instead we could on top of LeakSentry and attempt clearing things when LeakSentry lets us know there are leaking instances. That likely means having an intermediate report step where the sniper snoops in and reports and LeakCanary does not immediately dump / retries later. |
The viewgroup leak in P is definitely worth looking at fixing: #1081 (comment) |
Other option: https://twitter.com/ianhlake/status/1146793902336139264 |
Note: I went with the timer based approach rather than relying on signals from the app watcher because that way we don't have a direct dependency on it. |
This would be essentially the mirror of AndroidExcludedRef, a production library that uses various hacks to clean up framework leaks.
The text was updated successfully, but these errors were encountered: