commit 34677c7b0824ba84e11f9445c8c5503df2a8415d
author Mady Mellor <madym@google.com> Tue Jun 04 13:15:51 2024 -0700
committer Mady Mellor <madym@google.com> Tue Jun 04 20:25:11 2024 +0000
tree 6c78eebe1c093854493a6f77b96752578945b8ad
parent 38236869eec93ce71fd4a5ba13db2f56afc9ca0d

Maybe fix NPE - don't notifyChange if selected visibility is the same

Flag: none / bugfix
Test: none / can't repro
Test: manual - switch between the different radio buttons in bubble settings to make sure they
               all work
Bug: 343167554
Change-Id: I19fc3fa0df59c44e3f22de035c0b377caea1158f

src/com/android/settings/notification/app/BubblePreference.java

diff --git a/src/com/android/settings/notification/app/BubblePreference.java b/src/com/android/settings/notification/app/BubblePreference.java
index 29d6d30..17deef9 100644
--- a/src/com/android/settings/notification/app/BubblePreference.java
+++ b/src/com/android/settings/notification/app/BubblePreference.java
@@ -80,8 +80,10 @@
     }
 
     public void setSelectedVisibility(boolean visible) {
-        mSelectedVisible = visible;
-        notifyChanged();
+        if (mSelectedVisible != visible) {
+            mSelectedVisible = visible;
+            notifyChanged();
+        }
     }
 
     @Override