Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit cbec8517 authored by Jeff DeCew's avatar Jeff DeCew
Browse files

Add TODO about cleared media player notifications. 

Test: N/A
Change-Id: I1c6f86a89cf92989649a6b84b8012cc838bebcec
parent df90c681

packages/SystemUI/src/com/android/systemui/statusbar/notification/collection/NotifCollection.java

+1 −0
Original line number Diff line number Diff line
@@ -291,6 +291,7 @@ public class NotifCollection implements Dumpable { 
        mLogger.logDismissAll(userId);



        try {

            // TODO(b/169585328): Do not clear media player notifications

            mStatusBarService.onClearAllNotifications(userId);

        } catch (RemoteException e) {

            // system process is dead if we're here.

packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationStackScrollLayoutController.java

+3 −1
Original line number Diff line number Diff line
@@ -1301,7 +1301,8 @@ public class NotificationStackScrollLayoutController { 
        mView.clearNotifications(ROWS_GENTLE, closeShade);

    }



    private void onAnimationEnd(List<ExpandableNotificationRow> viewsToRemove, int selectedRows) {

    private void onAnimationEnd(List<ExpandableNotificationRow> viewsToRemove,

            @SelectedRows int selectedRows) {

        if (mFeatureFlags.isNewNotifPipelineRenderingEnabled()) {

            if (selectedRows == ROWS_ALL) {

                mNotifCollection.dismissAllNotifications(
@@ -1334,6 +1335,7 @@ public class NotificationStackScrollLayoutController { 
            }

            if (selectedRows == ROWS_ALL) {

                try {

                    // TODO(b/169585328): Do not clear media player notifications

                    mIStatusBarService.onClearAllNotifications(

                            mLockscreenUserManager.getCurrentUserId());

                } catch (Exception ignored) {