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

Commit 24598b6a authored by Abhijeet Kaur's avatar Abhijeet Kaur Committed by Automerger Merge Worker
Browse files

Merge "Delete empty bugreport files onError" into rvc-dev am: 873f59dc am: 8abcebf3

Change-Id: I00047412871cddec13d8921a4c4eb419dc4f9a78
parents 11fffe6f 8abcebf3
Loading
Loading
Loading
Loading
+17 −0
Original line number Diff line number Diff line
@@ -363,6 +363,7 @@ public class BugreportProgressService extends Service {
        public void onError(@BugreportErrorCode int errorCode) {
            synchronized (mLock) {
                stopProgressLocked(mInfo.id);
                mInfo.deleteEmptyFiles();
            }
            Log.e(TAG, "Bugreport API callback onError() errorCode = " + errorCode);
            return;
@@ -1979,6 +1980,22 @@ public class BugreportProgressService extends Service {
            bugreportFile.delete();
        }

        /**
         * Deletes empty files for a given bugreport.
         */
        private void deleteEmptyFiles() {
            if (bugreportFile.length() == 0) {
                Log.i(TAG, "Deleting empty bugreport file: " + bugreportFile);
                bugreportFile.delete();
            }
            for (File file : screenshotFiles) {
                if (file.length() == 0) {
                    Log.i(TAG, "Deleting empty screenshot file: " + file);
                    file.delete();
                }
            }
        }

        /**
         * Rename all screenshots files so that they contain the new {@code name} instead of the
         * {@code initialName} if user has changed it.