diff options
| author | 2023-01-11 14:27:10 +0000 | |
|---|---|---|
| committer | 2023-01-30 11:31:04 +0000 | |
| commit | d389823e9f144eadc57313c662ce3fd14e6e7382 (patch) | |
| tree | cac6d340c7b5d6e1b363fb60bdfc154991eff482 /libs/androidfw/ApkParsing.cpp | |
| parent | 16a472fa5ee5f922ded74c830c5e01f1c2c200fb (diff) | |
Cleanup in ExpandableNotificationRowTest
After adding some new test cases to ExpandableNotificationRowTest we
have noticed an increasing number of leaks of ExpandableNotificationRow.
How did this happen:
- Mockito was holding onto some ExpandableNotificationRow instances and resetting the inline mocks didn't help to release them
- NotificationTestHelper has a mock StatusBarStateController
- NotificationTestHelper creates a real HeadsUpManagerPhone passing in
the mock StatusBarStateController
- HeadsUpManagerPhone sets a listener on the mock StatusBarStateController
- that listener holds onto a NotificationEntry
- that entry has an ExpandableNotificationRow
This CL:
- uses a mock HeadsUpManagerPhone
- removes some unnecessary ExpandableNotificationRow instances from ExpandableNotificationRowTest
Fixes: 265135790
Test: create heap dumps using com.android.systemui.MemoryTrackingTestCase and compare them with go/ahat
Test: atest ExpandableNotificationRowTest
Test: atest SystemUITests
Change-Id: I314a1f6e59f095f8cc6018e3d65bf90bd9cd4a75
Diffstat (limited to 'libs/androidfw/ApkParsing.cpp')
0 files changed, 0 insertions, 0 deletions