diff options
| author | 2023-08-30 17:32:19 -0400 | |
|---|---|---|
| committer | 2023-09-12 15:13:03 +0000 | |
| commit | cf71e74f9ccde0100f053a897f40cc26a6394585 (patch) | |
| tree | 2473c37a0eacb90cf1b57b2ada662e9dba26b2bf /libs/hwui/HardwareBitmapUploader.cpp | |
| parent | a76fc6277a90d93d9f07084781ea9c9b9548f4f7 (diff) | |
Update Utils#isInService() to better reflect intent
Historically, the check defined in SettingsLib was like this. Consider a
service state to be IN_SERVICE if:
- The voice registration stae is IN_SERVICE or
- Voice registration is OOS/EMERGENCY_ONLY && the data registration
state is IN_SERVICE && !iwlan.
In the above logic, IWLAN was being checked for by querying the
NetworkRegistrationInfo from the service state for DOMAIN_PS +
TRANSPORT_TYPE_WLAN. If that registration was considered in service,
then we considered that network to be on IWLAN and would exclude it from
the overall inService check.
This CL makes a change to only check for the DOMAIN_PS +
TRANSPORT_TYPE_WWAN NetworkRegistrationInfo from the service state, and
checking specifically if _that_ reg info reports IN_SERVICE. Since WWAN
in this context means essentially a cellular connection rather than
wi-fi, this should be a simpler check for seeing if the cellular
connection is in service.
Test: UtilsTest.java
Bug: 288479957
Change-Id: I7dc9c7586858fae1a4ebb1e6f32057392547c39e
Diffstat (limited to 'libs/hwui/HardwareBitmapUploader.cpp')
0 files changed, 0 insertions, 0 deletions