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

Commit 0efc1844 authored by Garfield Tan's avatar Garfield Tan
Browse files

Don't tear down notification channels. 

Doing so will dismiss all notifications posted through it, including
warnings and failures.

Test: Failure notifications stay.
Bug: 62779990
Change-Id: I501fdc7f7ba9bf2205ef53d7c55ddfdf7a2d8fe7
parent ca4b65c1

src/com/android/documentsui/services/FileOperationService.java

+0 −8
Original line number Diff line number Diff line
@@ -180,8 +180,6 @@ public class FileOperationService extends Service implements Job.Listener { 
            Log.w(TAG, "Shutting down, but executor reports running jobs: " + unfinished);

        }



        tearDownNotificationChannel();



        executor = null;

        deletionExecutor = null;

        handler = null;
@@ -189,12 +187,6 @@ public class FileOperationService extends Service implements Job.Listener { 
        if (DEBUG) Log.d(TAG, "Destroyed.");

    }



    private void tearDownNotificationChannel() {

        if (features.isNotificationChannelEnabled()) {

            notificationManager.deleteNotificationChannel(NOTIFICATION_CHANNEL_ID);

        }

    }



    @Override

    public int onStartCommand(Intent intent, int flags, int serviceId) {

        // TODO: Ensure we're not being called with retry or redeliver.