DroidKaigi / conference-app-2019

The Official Conference App for DroidKaigi 2019 Tokyo
https://droidkaigi.jp/2019/en/
Apache License 2.0
816 stars 267 forks source link

Memory leaks in SessionDetailFragment onwards #839

Open surajsau opened 4 years ago

surajsau commented 4 years ago

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 commented 4 years ago

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

takahirom commented 4 years ago

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 commented 4 years ago

あ! ι–“ι•γ„γΎγ—γŸοΌγ”γ‚γ‚“γͺさい!笑笑