Android Studio Memory Profiler heap dump does not show total retained size for classes [282053096]

Bug

Status Update

No update yet.

Description

lo...@applovin.com

created issue #1

May 12, 2023 08:51AM

See example in screenshot attached. "Retained Size" for "String" is very low, despite the retained size for individual string objects being very high. In the past, this column would show the total retained size for all strings.

Also attached the hprof file for the heap dump in the screenshot for reference.

```
Build: AI-222.4459.24.2221.9971841, 202304191751,

AI-222.4459.24.2221.9971841, JRE 17.0.6+0-17.0.6b802.4-9586694x64 JetBrains s.r.o., OS Mac OS X(aarch64) v13.2.1, screens 3456.0x2234.0; Retina

AS: Flamingo | 2022.2.1 Patch 1
Kotlin plugin: 222-1.8.20-release-AS3739.54
Android Gradle Plugin: 3.6.0
Gradle: 6.1.1
Gradle JDK: Azul Zulu version 13.0.14
NDK: from local.properties: (not specified), latest from SDK: (not found)
CMake: from local.properties: (not specified), latest from SDK: (not found), from PATH: (not found)
Source: send_feedback_icon

IMPORTANT: Please read

https://developer.android.com/studio/report-bugs.html carefully and supply all required information.
```

Screenshot 2023-05-12 at 1.44.29 AM.png

898 KB

View

Download

wrong_retained_size.hprof

60 MB

Download

Comments

di...@google.com <di...@google.com> May 12, 2023 10:05AM

Assigned to an...@google.com.

sh...@google.com <sh...@google.com> May 17, 2023 06:15PM

Status: New

sh...@google.com <sh...@google.com> #2May 19, 2023 06:53PM

Any update on this?
The request is DiffUtil implementation along with "items" and "indexedItems" functions to support DiffUtil and animations when adding/removing items. the same as the ListAdapter with DiffUtil.

Message last modified on May 19, 2023 06:56PM

ow...@gmail.com <ow...@gmail.com> #3Jun 16, 2023 10:35PM

We are working on it, but nothing to share yet. Thanks

sh...@google.com <sh...@google.com> #4Jun 16, 2023 10:45PM

Are we allowed to post potential workarounds on issue trackers? I created this small POC repo, definitely not a solution and probably not ideal in the long run but works for my use case and thought I'd share: https://github.com/RoudyK/AnimatedLazyColumn (Please delete my comment or let me know so I can delete it, in case I'm not allowed to post workarounds)

lo...@applovin.com <lo...@applovin.com> #5Jul 11, 2023 06:51PM

I think there is not much point in coming up with workaround as this is expected to be one of the main features/supports for the DiffUtil (list diffing for adapters). Entire RecyclerView implementations depend on this.

Issue 282053096