Skip to content
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

Memory leaks in SessionDetailFragment onwards #839

Open
surajsau opened this issue Jan 15, 2020 · 3 comments
Open

Memory leaks in SessionDetailFragment onwards #839

surajsau opened this issue Jan 15, 2020 · 3 comments

Comments

@surajsau
Copy link

surajsau commented Jan 15, 2020

Overview:
When session details screen is opened, frequent memory leaks are captured by LeakCanary

Steps To Reproduce:
launch app
click on any session
scroll up and down or click on feedback and scroll between the views
Leak Canary records memory leaks very frequently.

Expected behavior:
Memory leaks must be reduced

Env:

Device: [e.g. Pixel3]
OS: [e.g. 9.0]
App Version or the branch [master]

Additional context:
It's happening because of ViewDataBinding. It isn't being cleared in onDestroyView()
We can take out binding as variable in the base DaggerFragment and set it to null in onDestroyView()

Screenshot 2020-01-15 at 1 50 22 PM

Screenshot 2020-01-15 at 1 49 52 PM

@surajsau
Copy link
Author

@takahirom I can take it up and fix it if needed.

@takahirom
Copy link
Member

Thank you!
But, now we are working for DroidKaigi/conference-app-2020. This repository is conference-app-2019
If you don't care, please take a look at the 2020 repositories 🙏
https://github.com/DroidKaigi/conference-app-2020

@surajsau
Copy link
Author

あ! 間違いました!ごめんなさい!笑笑

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants