commit 12066c72f95a0ec0484370f00d216c6b870f4cef
author SongFerngWang <songferngwang@google.com> Fri Mar 17 18:36:07 2023 +0800
committer SongFerngWang <songferngwang@google.com> Fri Mar 17 18:49:38 2023 +0800
tree ee6410a74c831df170d81b7cb5b45c00d8714ab9
parent 813f654aa12c422ba16697761b374d07ca4782f9

Update OpenNetworkSelect UI at main thread

Bug: 270652395
Test: local test and atest AutoSelectPreferenceControllerTest
Change-Id: Idfc7a07106d552c35a94414bb14eac0fbdc3974f

src/com/android/settings/network/telephony/gsm/AutoSelectPreferenceController.java

diff --git a/src/com/android/settings/network/telephony/gsm/AutoSelectPreferenceController.java b/src/com/android/settings/network/telephony/gsm/AutoSelectPreferenceController.java
index f46a452..c0b903d 100644
--- a/src/com/android/settings/network/telephony/gsm/AutoSelectPreferenceController.java
+++ b/src/com/android/settings/network/telephony/gsm/AutoSelectPreferenceController.java
@@ -211,6 +211,7 @@
                             mSwitchPreference.setChecked(isChecked());
                         }
                         mRecursiveUpdate.decrementAndGet();
+                        updateListenerValue();
                         dismissProgressBar();
                     });
                 });
@@ -245,13 +246,13 @@
 
         ThreadUtils.postOnBackgroundThread(() -> {
             queryNetworkSelectionMode(INTERNAL_LOG_TAG_INIT);
-
             //Update UI in UI thread
             mUiHandler.post(() -> {
                 if (mSwitchPreference != null) {
                     mRecursiveUpdate.getAndIncrement();
                     mSwitchPreference.setChecked(isChecked());
                     mRecursiveUpdate.decrementAndGet();
+                    updateListenerValue();
                 }
             });
         });
@@ -267,7 +268,6 @@
     private void queryNetworkSelectionMode(String tag) {
         mCacheOfModeStatus = mTelephonyManager.getNetworkSelectionMode();
         Log.d(LOG_TAG, tag + ": query commend done. mCacheOfModeStatus: " + mCacheOfModeStatus);
-        updateListenerValue();
     }
 
     @VisibleForTesting
@@ -284,12 +284,11 @@
             }
             mCacheOfModeStatus = networkSelectionMode;
             Log.d(LOG_TAG, "updateUiAutoSelectValue: mCacheOfModeStatus: " + mCacheOfModeStatus);
-            updateListenerValue();
 
             mRecursiveUpdate.getAndIncrement();
             updateState(mSwitchPreference);
             mRecursiveUpdate.decrementAndGet();
-
+            updateListenerValue();
         }
     }