Make sure to update config when setBubblesAllowed is called for apps

Bug: 130441629
Test: atest PreferencesHelperTest
Change-Id: Ib1558a99b549ae05f2b361259b1fe840b667f12e

services/core/java/com/android/server/notification/PreferencesHelper.java

@@ -477,11 +477,16 @@ public class PreferencesHelper implements RankingConfig {
      * @param allowed whether bubbles are allowed.
      */
     public void setBubblesAllowed(String pkg, int uid, boolean allowed) {
+        boolean changed = false;
         synchronized (mPackagePreferences) {
             PackagePreferences p = getOrCreatePackagePreferencesLocked(pkg, uid);
+            changed = p.allowBubble != allowed;
             p.allowBubble = allowed;
             p.lockedAppFields = p.lockedAppFields | LockableAppFields.USER_LOCKED_BUBBLE;
         }
+        if (changed) {
+            updateConfig();
+        }
     }
 
     /**

services/tests/uiservicestests/src/com/android/server/notification/PreferencesHelperTest.java

@@ -2676,4 +2676,11 @@ public class PreferencesHelperTest extends UiServiceTestCase {
         assertTrue(mHelper.getNotificationChannel(PKG_O, UID_O, channel1.getId(), false)
                 .isImportanceLockedByCriticalDeviceFunction());
     }
+
+    @Test
+    public void testSetBubblesAllowed_false() {
+        mHelper.setBubblesAllowed(PKG_O, UID_O, false);
+        assertFalse(mHelper.areBubblesAllowed(PKG_O, UID_O));
+        verify(mHandler, times(1)).requestSort();
+    }
 }