Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Add a recycle bitmap check before extracting wallpaper color" into tm-qpr-dev

Browse Source
This commit is contained in:
Michael Mikhail
2023-03-03 19:43:23 +00:00
committed by Android (Google) Code Review
parent dcef87059b 2f4d0ea966
commit aee5a4332a
2 changed files with 20 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
packages/SystemUI/src/com/android/systemui/media/controls/ui/MediaControlPanel.java
View File

@@ -928,13 +928,16 @@ public class MediaControlPanel {
if (artworkIcon.getType() == Icon.TYPE_BITMAP
|| artworkIcon.getType() == Icon.TYPE_ADAPTIVE_BITMAP) {
// Avoids extra processing if this is already a valid bitmap
return WallpaperColors
.fromBitmap(artworkIcon.getBitmap());
Bitmap artworkBitmap = artworkIcon.getBitmap();
if (artworkBitmap.isRecycled()) {
Log.d(TAG, "Cannot load wallpaper color from a recycled bitmap");
return null;
}
return WallpaperColors.fromBitmap(artworkBitmap);
} else {
Drawable artworkDrawable = artworkIcon.loadDrawable(mContext);
if (artworkDrawable != null) {
return WallpaperColors
.fromDrawable(artworkIcon.loadDrawable(mContext));
return WallpaperColors.fromDrawable(artworkDrawable);
}
}
}

13
packages/SystemUI/tests/src/com/android/systemui/media/controls/ui/MediaControlPanelTest.kt
View File

@@ -730,6 +730,19 @@ public class MediaControlPanelTest : SysuiTestCase() {
.isNotEqualTo(greenArtwork.getDrawable(1).constantState)
}
@Test
fun getWallpaperColor_recycledBitmap_notCrashing() {
// Setup redArt icon.
val redBmp = Bitmap.createBitmap(10, 10, Bitmap.Config.ARGB_8888)
val redArt = Icon.createWithBitmap(redBmp)
// Recycle bitmap of redArt icon.
redArt.bitmap.recycle()
// get wallpaperColor without illegal exception.
player.getWallpaperColor(redArt)
}
@Test
fun bind_seekBarDisabled_hasActions_seekBarVisibilityIsSetToInvisible() {
useRealConstraintSets()