Memory leak in PreferenceFragmentCompat adapter handling [121006469]

Fixed

Bug

Status Update

No update yet.

Description

js...@gmail.com

created issue #1

Dec 14, 2018 07:54PM

RecyclerView registers an observer (

https://android.googlesource.com/platform/frameworks/support/+/androidx-master-dev/recyclerview/recyclerview/src/main/java/androidx/recyclerview/widget/RecyclerView.java#1201) that is only unregistered manually, by either setting the adapter again, or calling unregisterAdapterDataObserver externally.

PreferenceFragmentCompat sets this adapter (PreferenceGroupAdapter), which implicitly registers an observer. PreferenceFragmentCompat never sets the adapter to null when the view is destroyed, so the observer keeps references to the RecyclerView and doesn't get gc'ed.

I've worked around this by explicitly setting the adapter to null in an override of onDestroyView, but since my component doesn't explicitly have ownership of the list handling, I'd argue that PreferenceFragmentCompat should do this cleanup.

It would be solved by setting the RecyclerView adapter to null in the fragment's onDestroyView.

Here's a memory dump.

2018-12-14_10-38-54_921.hprof

23 MB

Download

Comments

lp...@google.com <lp...@google.com> #2Jan 10, 2019 11:35PM

Project: platform/frameworks/support
Branch: androidx-main

commit 20112348e9cf68cf81ae9b54fb3c0b18150edb71
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Fri May 10 11:10:17 2024

Move `compose-ui` to `androidMain` in `viewmodel-compose`

Test: N/A
Fixes: 339562627
Change-Id: Ia3b0638b3f4b61ba760d3c10337275a255d898e8

M lifecycle/lifecycle-viewmodel-compose/build.gradle

https://android-review.googlesource.com/3079334

lp...@google.com <lp...@google.com> #3Jan 14, 2019 08:41PM

Marked as fixed.

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.lifecycle:lifecycle-viewmodel-compose:2.8.1
androidx.lifecycle:lifecycle-viewmodel-compose-android:2.8.1
androidx.lifecycle:lifecycle-viewmodel-compose-desktop:2.8.1

ma...@gmail.com <ma...@gmail.com> #4Feb 28, 2020 01:26PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.lifecycle:lifecycle-viewmodel-compose:2.9.0-alpha01
androidx.lifecycle:lifecycle-viewmodel-compose-android:2.9.0-alpha01
androidx.lifecycle:lifecycle-viewmodel-compose-desktop:2.9.0-alpha01

jb...@google.com <jb...@google.com> #5Feb 28, 2020 05:57PM

Since the Fragment 1.2.0-alpha02 release fragments wait until exit animations are complete before onDestroyView() is called so this should not be an issue in any current Fragment versions.

Please file a bug against Fragments the with a sample project that reproduces this issue and we will be happy to take a look at what's going on.

ma...@gmail.com <ma...@gmail.com> #6Feb 29, 2020 09:16PM

I can confirm that using fragment 1.2.2 resolves the issue, thank you!

Without an explicit dependency declaration version 1.1.0 was still used (transient dependency of appcompat 1.1.0).

Issue 121006469