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

Commit d5acaee7 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Ignore nonsensical clears from NoMan" into rvc-dev am: 191d081b am:... 

Merge "Ignore nonsensical clears from NoMan" into rvc-dev am: 191d081b am: d0f09fc3 am: 9713dbfb am: 7b3f3e7c

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

Change-Id: If6f7fdb51f73d082856d3f3833c9a544966f84fb
parents a1667cb8 7b3f3e7c

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

+2 −3
Original line number Diff line number Diff line
@@ -382,9 +382,8 @@ public class NotifCollection implements Dumpable { 


        final NotificationEntry entry = mNotificationSet.get(sbn.getKey());

        if (entry == null) {

            crashIfNotInitializing(

                    new IllegalStateException("No notification to remove with key "

                            + sbn.getKey()));

            // TODO (b/160008901): Throw an exception here

            mLogger.logNoNotificationToRemoveWithKey(sbn.getKey());

            return;

        }

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

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

    }



    fun logNoNotificationToRemoveWithKey(key: String) {

        buffer.log(TAG, ERROR, {

            str1 = key

        }, {

            "No notification to remove with key $str1"

        })

    }



    fun logRankingMissing(key: String, rankingMap: RankingMap) {

        buffer.log(TAG, WARNING, { str1 = key }, { "Ranking update is missing ranking for $str1" })

        buffer.log(TAG, DEBUG, {}, { "Ranking map contents:" })

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

+4 −3
Original line number Diff line number Diff line
@@ -1273,8 +1273,8 @@ public class NotifCollectionTest extends SysuiTestCase { 
        verify(mInterceptor3, never()).shouldInterceptDismissal(clearable);

    }



    @Test(expected = IllegalStateException.class)

    public void testClearNotificationThrowsIfMissing() {

    @Test

    public void testClearNotificationDoesntThrowIfMissing() {

        // GIVEN that enough time has passed that we're beyond the forgiveness window

        mClock.advanceTime(5001);
@@ -1287,7 +1287,8 @@ public class NotifCollectionTest extends SysuiTestCase { 
                container.getSbn(),

                new RankingMap(new Ranking[]{ container.getRanking() }));



        // THEN an exception is thrown

        // THEN the event is ignored

        verify(mCollectionListener, never()).onEntryRemoved(any(NotificationEntry.class), anyInt());

    }



    @Test