[SB][Chips] Update call chip to use icon from notification.
This changes updates the ongoing phone call chip to use the `Notification.getSmallIcon()` as the icon inside the chip, instead of hard-coding it to be a phone icon. This includes: 1) For each notification, creating a new `statusBarChipIcon` in addition to creating the other location-specific icons (status bar, shelf, aod) 2) Updating the paddings in the chip layout, since the notification icon is required to come with its own padding Bug: 354930838 Flag: NONE required for imminent release Test: Start call without timer -> verify chip has icon only, icon is centered Test: Start ongoing call -> verify chip has icon and time Test: Chip UI at various font and display sizes Test: Theme changes (chip text updates to both white and black correctly) Test: Accessibility for chip Test: Enable com.android.systemui.status_bar_screen_sharing_chips flag -> verify call and screen chips still work okay (the paddings are off but general functionality still works) Test: atest OngoingCallControllerTest Change-Id: Id7ad51ad3a590e6556987cba4f2639870c422f92 Merged-In: Id7ad51ad3a590e6556987cba4f2639870c422f92
Showing
- packages/SystemUI/res/layout/ongoing_activity_chip.xml 5 additions, 6 deletionspackages/SystemUI/res/layout/ongoing_activity_chip.xml
- packages/SystemUI/res/values/dimens.xml 6 additions, 0 deletionspackages/SystemUI/res/values/dimens.xml
- packages/SystemUI/src/com/android/systemui/statusbar/notification/icon/IconManager.kt 11 additions, 1 deletion...droid/systemui/statusbar/notification/icon/IconManager.kt
- packages/SystemUI/src/com/android/systemui/statusbar/notification/icon/IconPack.java 18 additions, 2 deletions...ndroid/systemui/statusbar/notification/icon/IconPack.java
- packages/SystemUI/src/com/android/systemui/statusbar/phone/ongoingcall/OngoingCallController.kt 56 additions, 4 deletions...emui/statusbar/phone/ongoingcall/OngoingCallController.kt
- packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/ongoingcall/OngoingCallControllerTest.kt 135 additions, 0 deletions.../statusbar/phone/ongoingcall/OngoingCallControllerTest.kt
Loading
Please register or sign in to comment