diff options
author | Presubmit Automerger Backend <android-build-presubmit-automerger-backend@system.gserviceaccount.com> | 2022-02-14 20:37:05 +0000 |
---|---|---|
committer | Presubmit Automerger Backend <android-build-presubmit-automerger-backend@system.gserviceaccount.com> | 2022-02-14 20:37:05 +0000 |
commit | b812cccb379e0cd2cde11fcadd4fdc3e630b7d8b (patch) | |
tree | bc9d913b7f9a018b9df9c3d492f0faa9e73e0a1c /src/com/android/cellbroadcastreceiver/CellBroadcastSettings.java | |
parent | d1cbaf490459b5a06ab77921d7ad77f4bb6171b9 (diff) | |
parent | 46011e24805e2d608caa7eab56ad7dcf2ef1a4c5 (diff) | |
download | CellBroadcastReceiver-b812cccb379e0cd2cde11fcadd4fdc3e630b7d8b.tar.gz |
[automerge] Remove the redundant cache of resources 2p: 46011e2480
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/CellBroadcastReceiver/+/16854983
Bug: 219549062
Change-Id: I0ada38e927b1d77a026da0ff60fc91f58ccfe0c2
Diffstat (limited to 'src/com/android/cellbroadcastreceiver/CellBroadcastSettings.java')
-rw-r--r-- | src/com/android/cellbroadcastreceiver/CellBroadcastSettings.java | 20 |
1 files changed, 1 insertions, 19 deletions
diff --git a/src/com/android/cellbroadcastreceiver/CellBroadcastSettings.java b/src/com/android/cellbroadcastreceiver/CellBroadcastSettings.java index aae899d43..9f609d42e 100644 --- a/src/com/android/cellbroadcastreceiver/CellBroadcastSettings.java +++ b/src/com/android/cellbroadcastreceiver/CellBroadcastSettings.java @@ -158,9 +158,6 @@ public class CellBroadcastSettings extends CollapsingToolbarBaseActivity { /* End of user preferences keys section. */ - // Resource cache - private static final Map<Integer, Resources> sResourcesCache = new HashMap<>(); - // Resource cache per operator private static final Map<String, Resources> sResourcesCacheByOperator = new HashMap<>(); private static final Object sCacheLock = new Object(); @@ -858,21 +855,7 @@ public class CellBroadcastSettings extends CollapsingToolbarBaseActivity { return context.getResources(); } - synchronized (sCacheLock) { - if (sResourcesCache.containsKey(subId)) { - return sResourcesCache.get(subId); - } - - Resources res = SubscriptionManager.getResourcesForSubId(context, subId); - - if (res.getConfiguration().mnc != 0) { - Log.d(TAG, "Cache resource for sub: " + subId + ", mcc: " - + res.getConfiguration().mcc + ", mnc:" + res.getConfiguration().mnc); - sResourcesCache.put(subId, res); - } - - return res; - } + return SubscriptionManager.getResourcesForSubId(context, subId); } /** @@ -973,7 +956,6 @@ public class CellBroadcastSettings extends CollapsingToolbarBaseActivity { public static void resetResourcesCache() { synchronized (sCacheLock) { sResourcesCacheByOperator.clear(); - sResourcesCache.clear(); } } } |