Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "[SB Refactor] Don't display the wifi icon if wifi isn't default." into tm-qpr-dev am: c23daaa7a4 am: 2fe46e157e

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/21937135

Change-Id: I8c7c03dd188d57957f2c3ba7c6cb513dc7a9dc5a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Caitlin Shkuratov
2023-03-13 15:31:04 +00:00
committed by Automerger Merge Worker
parent aca621c242 2fe46e157e
commit b755b28b94
2 changed files with 13 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
packages/SystemUI/src/com/android/systemui/statusbar/pipeline/wifi/ui/viewmodel/WifiViewModel.kt
View File

@@ -124,7 +124,8 @@ constructor(
isDefault -> icon
wifiConstants.alwaysShowIconIfEnabled -> icon
!connectivityConstants.hasDataCapabilities -> icon
wifiNetwork is WifiNetworkModel.Active && wifiNetwork.isValidated -> icon
// See b/272509965: Even if we have an active and validated wifi network, we
// don't want to show the icon if wifi isn't the default network.
else -> WifiIcon.Hidden
}
}

25
packages/SystemUI/tests/src/com/android/systemui/statusbar/pipeline/wifi/ui/viewmodel/WifiViewModelIconParameterizedTest.kt
View File

@@ -368,40 +368,37 @@ internal class WifiViewModelIconParameterizedTest(private val testCase: TestCase
// network = CarrierMerged => not shown
TestCase(
enabled = true,
isDefault = true,
forceHidden = false,
network =
WifiNetworkModel.CarrierMerged(NETWORK_ID, subscriptionId = 1, level = 1),
expected = null,
),
// network = Inactive => not shown
// isDefault = false => no networks shown
TestCase(
isDefault = false,
network = WifiNetworkModel.Inactive,
expected = null,
),
// network = Unavailable => not shown
TestCase(
isDefault = false,
network = WifiNetworkModel.Unavailable,
expected = null,
),
// network = Active & validated = false => not shown
TestCase(
isDefault = false,
network = WifiNetworkModel.Active(NETWORK_ID, isValidated = false, level = 3),
expected = null,
),
// network = Active & validated = true => shown
// Even though this network is active and validated, we still doesn't want it shown
// because wifi isn't the default connection (b/272509965).
TestCase(
isDefault = false,
network = WifiNetworkModel.Active(NETWORK_ID, isValidated = true, level = 4),
expected =
Expected(
iconResource = WIFI_FULL_ICONS[4],
contentDescription = { context ->
context.getString(WIFI_CONNECTION_STRENGTH[4])
},
description = "Full internet level 4 icon",
),
expected = null,
),
)
}