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

Commit c254b0fe authored by Jeff DeCew's avatar Jeff DeCew Committed by Automerger Merge Worker
Browse files

Merge changes Id9f50e84,I4d7f0497 into tm-qpr-dev am: 85fc0ce3 am: 5d4f557b 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/18379004



Change-Id: I1a1a05ae19c0449720dd378ca04cbb4f82703405
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents f801da6b 5d4f557b

packages/SystemUI/src/com/android/systemui/statusbar/notification/collection/NotifCollection.java

+13 −4
Original line number Diff line number Diff line
@@ -609,12 +609,21 @@ public class NotifCollection implements Dumpable { 
        }

        checkForReentrantCall();



        NotificationEntry collectionEntry = getEntry(entry.getKey());

        String logKey = logKey(entry);

        String collectionEntryIs = collectionEntry == null ? "null"

                : entry == collectionEntry ? "same" : "different";



        if (entry != collectionEntry) {

            // TODO: We should probably make this throw, but that's too risky right now

            mLogger.logEntryBeingExtendedNotInCollection(entry, extender, collectionEntryIs);

        }



        if (!entry.mLifetimeExtenders.remove(extender)) {

            throw mEulogizer.record(new IllegalStateException(

                    String.format(

                            "Cannot end lifetime extension for extender \"%s\" (%s)",

                            extender.getName(),

                            extender)));

                    String.format("Cannot end lifetime extension for extender \"%s\""

                                    + " of entry %s (collection entry is %s)",

                            extender.getName(), logKey, collectionEntryIs)));

        }



        mLogger.logLifetimeExtensionEnded(

packages/SystemUI/src/com/android/systemui/statusbar/notification/collection/notifcollection/NotifCollectionLogger.kt

+14 −0
Original line number Diff line number Diff line
@@ -248,6 +248,20 @@ class NotifCollectionLogger @Inject constructor( 
        })

    }



    fun logEntryBeingExtendedNotInCollection(

        entry: NotificationEntry,

        extender: NotifLifetimeExtender,

        collectionEntryIs: String

    ) {

        buffer.log(TAG, WARNING, {

            str1 = entry.logKey

            str2 = extender.name

            str3 = collectionEntryIs

        }, {

            "While ending lifetime extension by $str2 of $str1, entry in collection is $str3"

        })

    }



    fun logFutureDismissalReused(dismissal: FutureDismissal) {

        buffer.log(TAG, INFO, {

            str1 = dismissal.label