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

Omit notification small icons from history parcel

Browse Source 
They aren't shown in the UI, so we can save some work

Test: load notification history page
Fixes: 206973316
Change-Id: Ia5089bd2f051e40f242f9f72528cbb8251ad9684
This commit is contained in:
Julia Reynolds
2021-12-08 10:22:00 -05:00
parent b3f9a846ec
commit 589987ce20
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
core/java/android/app/NotificationHistory.java
View File

@@ -497,7 +497,11 @@ public final class NotificationHistory implements Parcelable {
p.writeLong(notification.getPostedTimeMs());
p.writeString(notification.getTitle());
p.writeString(notification.getText());
notification.getIcon().writeToParcel(p, flags);
p.writeBoolean(false);
// The current design does not display icons, so don't bother adding them to the parcel
//if (notification.getIcon() != null) {
// notification.getIcon().writeToParcel(p, flags);
//}
}
/**
@@ -539,7 +543,9 @@ public final class NotificationHistory implements Parcelable {
notificationOut.setPostedTimeMs(p.readLong());
notificationOut.setTitle(p.readString());
notificationOut.setText(p.readString());
notificationOut.setIcon(Icon.CREATOR.createFromParcel(p));
if (p.readBoolean()) {
notificationOut.setIcon(Icon.CREATOR.createFromParcel(p));
}
return notificationOut.build();
}