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

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

Show predefined title and description in finished notification

shareTitle and shareDescription is passed by the caller of the
bugreport. Show these in the finished bugreport notification.

Pre-set shareTitle takes precedence over user modified title.

shareDescription and description are not related/dependant on each other
in any way.

Bug: 150333444
Test: Manual (by passing EXTRA_TITLE and EXTRA_DESCRIPTION from
ActivityManagerService when trigerring a bugreport)

Change-Id: I2bfd080aeee677cdc8d0af339d7ad4a29451c3e0
parent 09b1ee60
Loading
Loading
Loading
Loading
+7 −3
Original line number Diff line number Diff line
@@ -1019,6 +1019,9 @@ public class BugreportProgressService extends Service {
     * Wraps up bugreport generation and triggers a notification to share the bugreport.
     */
    private void onBugreportFinished(BugreportInfo info) {
        if (!TextUtils.isEmpty(info.shareTitle)) {
            info.setTitle(info.shareTitle);
        }
        Log.d(TAG, "Bugreport finished with title: " + info.getTitle()
                + " and shareDescription: " + info.shareDescription);
        info.finished = new AtomicBoolean(true);
@@ -1795,7 +1798,9 @@ public class BugreportProgressService extends Service {

        /**
         * User-provided, detailed description of the bugreport; when set, will be added to the body
         * of the {@link Intent#ACTION_SEND_MULTIPLE} intent.
         * of the {@link Intent#ACTION_SEND_MULTIPLE} intent. This is shown in the app where the
         * bugreport is being shared as an attachment. This is not related/dependant on
         * {@code shareDescription}.
         */
        private String description;

@@ -2130,7 +2135,6 @@ public class BugreportProgressService extends Service {
                return new BugreportInfo[size];
            }
        };

    }

    @GuardedBy("mLock")