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

Commit dd58deea authored by Tetiana Meronyk's avatar Tetiana Meronyk
Browse files

Remove unused field after flag cleanup 

int mNotificationClientUid was used when this class could handle one notification at a time. With the implementation of handling simultaneous alarms it has become unused.

Also adding @SuppressLint annotation to a private method to stop the warning.

Bug: 432469009
Test: manual
Flag: EXEMPT removing unused code
Change-Id: I88c89510c81b25ca62d305238e964dacd52b70fb
parent 80b5b068

services/core/java/com/android/server/pm/BackgroundUserSoundNotifier.java

+1 −8
Original line number Diff line number Diff line
@@ -59,14 +59,6 @@ public class BackgroundUserSoundNotifier { 
            "com.android.server.ACTION_DISMISS_NOTIFICATION";

    private static final String EXTRA_NOTIFICATION_CLIENT_UID =

            "com.android.server.EXTRA_CLIENT_UID";

    /**

     * The clientUid from the AudioFocusInfo of the background user,

     * for which an active notification is currently displayed.

     * Set to -1 if no notification is being shown.

     * TODO: b/367615180 - add support for multiple simultaneous alarms

     */

    @VisibleForTesting

    int mNotificationClientUid = -1;

    /**

     * UIDs of audio focus infos with active notifications.

     */
@@ -102,6 +94,7 @@ public class BackgroundUserSoundNotifier { 
        mNotificationManager.createNotificationChannel(channel);

    }



    @SuppressLint("MissingPermission")

    private void setupFocusControlAudioPolicy() {

        // Used to configure our audio policy to handle focus events.

        // This gives us the ability to decide which audio focus requests to accept and bypasses