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

Browse Source
This commit is contained in:
Caitlin Shkuratov
2023-03-13 14:23:59 +00:00
committed by Android (Google) Code Review
parent c2936af3cf 581ae211a2
commit c23daaa7a4
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,
),
)
}