I am experiencing a strange memory leak with Compose with An kotlinlang #compose

I am experiencing a strange memory leak with Compose with AnimatedNavHost from Accompanist and AndroidView. 1. Start from page A that contains an AndroidView 2. Navigate to page B (without popping page A) 3. (Page A is removed from the composition (onDispose called etc)) 4. Page B is popped off the stack, returning to A 5. Repeat indefinitely. Every time page A is displayed, a new View is created and the previous Views never get garbage collected. Any idea why this might be happening? Is it possible that a reference to the View could be accidentally stored somewhere or could this be a bug in Compose/Accompanist/Navigation? An additional possibly important detail: GC happens normally if A is popped off the navigation stack. EDIT: I managed to debug this and find the root cause. ~~I will write an explanation in~~ 🧵 ~~later if anyone else experiences the same issue.~~ Done.

On the page A that was leaking memory, we had nested content like this: • Compose(1) -> AndroidView -> RecyclerView -> Compose(2) It turns out Compose doesn't release references when detaching from window if the content is inside a RecyclerView, probably because it will likely be reused soon. So compose indirectly kept a reference to the RecyclerView and all the content inside, which was causing the memory leak when new views were however being created on every navigation. We fixed this by manually calling

.callPoolingContainerOnRelease()

from https://developer.android.com/jetpack/androidx/releases/customview#customview-poolingcontainer-1.0.0 on the

onDetachedFromWindow()

of the RecyclerView. This is actually mentioned in the documentation as well: https://developer.android.com/jetpack/compose/migrate/interoperability-apis/compose-in-views#composition-strategy

[...] In some of these situations, the app can also slowly leak memory from Composition instances unless you manually call
AbstractComposeView.disposeComposition
.

What however still puzzles me is why the GC succeeded when the page was popped off the navigation stack (at least sometimes) 🤔

Zach Klippenstein (he/him) [MOD]

03/21/2023, 3:34 PM

Cc @Ryan Mentley

39 Views

#compose Join Slack