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

Commit 0d61c4f8 authored by Abhijeet Kaur's avatar Abhijeet Kaur
Browse files

Leave error handling the same as old workflow using dumpstate 

Removing the TODO as won't be working on improving error handling as
part of this bug. Created another bug (b/138576498) to work on better
error handling.

Bug: 123617758
Bug: 127431371
Test: builds (only removing comment)
Change-Id: I8a7fea416e9871909cfb3cc801232bff6056536e
parent d94c8e72

packages/Shell/src/com/android/shell/BugreportProgressService.java

+4 −2
Original line number Diff line number Diff line
@@ -365,8 +365,10 @@ public class BugreportProgressService extends Service { 
            checkProgressUpdated(mInfo, (int) progress);

        }



        // TODO(b/127431371): Add error code handling for bugreport API errors.

        // Logging errors and removing progress notification for now.

        /**

         * Logs errors and stops the service on which this bugreport was running.

         * Also stops progress notification (if any).

         */

        @Override

        public void onError(@BugreportErrorCode int errorCode) {

            stopProgress(mId);