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

Commit 7b3f3e7c 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

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

Change-Id: Ic9081c2bc3aef3b443690fbd5aef8ce0896227d8
parents dde6cc46 9713dbfb

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