43ed6f45edb2395f3dddbe2b4602c14debbd6e02 - platform/frameworks/base

commit	43ed6f45edb2395f3dddbe2b4602c14debbd6e02	[log] [tgz]
author	Malcolm Chen <refuhoo@google.com>	Tue Aug 13 13:24:07 2019 -0700
committer	Brad Ebinger <breadley@google.com>	Thu Jan 23 13:02:52 2020 -0800
tree	6c1f9f5ff271c929e9912889517621e8d4e130fb
parent	0ed49867e2a02590c066dd4b22a8fc5b091932d7 [diff]

Move filterMobileSubscriptionInSameGroup into KeyguardUpdateMonitor.

This avoids race condition that, KeyguardUpdateMonitor refreshes active
sub list however CarrierTextController filtered it based on stale active
data subId.

By move filterMobileSubscriptionInSameGroup into KeyguardUpdateMonitor,
it becomes the single source of truth about which carrier name should be
shown.

Bug: 138456731
Test: manual and unittest
Merged-In: Ida3a90c5408b056cad7a79fbba84fe880ce1e5be
Change-Id: Ida3a90c5408b056cad7a79fbba84fe880ce1e5be

packages/SystemUI/src/com/android/keyguard/CarrierTextController.java[diff]
packages/SystemUI/src/com/android/keyguard/KeyguardUpdateMonitor.java[diff]
packages/SystemUI/src/com/android/systemui/statusbar/OperatorNameView.java[diff]
packages/SystemUI/src/com/android/systemui/statusbar/policy/EmergencyCryptkeeperText.java[diff]
packages/SystemUI/tests/src/com/android/keyguard/CarrierTextControllerTest.java[diff]
packages/SystemUI/tests/src/com/android/keyguard/KeyguardUpdateMonitorTest.java[diff]

6 files changed