Merge "Make disabledProviderList nullable"

This commit is contained in:
Qinmei Du
2022-12-01 16:59:14 +00:00
committed by Android (Google) Code Review
2 changed files with 6 additions and 6 deletions

View File

@@ -59,7 +59,7 @@ class CredentialManagerRepo(
) {
val requestInfo: RequestInfo
private val providerEnabledList: List<ProviderData>
private val providerDisabledList: List<DisabledProviderData>
private val providerDisabledList: List<DisabledProviderData>?
// TODO: require non-null.
val resultReceiver: ResultReceiver?
@@ -143,7 +143,7 @@ class CredentialManagerRepo(
providerEnabledList as List<CreateCredentialProviderData>, requestDisplayInfo, context)
val providerDisabledList = CreateFlowUtils.toDisabledProviderList(
// Handle runtime cast error
providerDisabledList as List<DisabledProviderData>, context)
providerDisabledList, context)
var defaultProvider: EnabledProviderInfo? = null
var remoteEntry: RemoteInfo? = null
providerEnabledList.forEach{providerInfo -> providerInfo.createOptions =
@@ -209,7 +209,7 @@ class CredentialManagerRepo(
)
}
private fun testDisabledProviderList(): List<DisabledProviderData> {
private fun testDisabledProviderList(): List<DisabledProviderData>? {
return listOf(
DisabledProviderData("com.lastpass.lpandroid"),
DisabledProviderData("com.google.android.youtube")

View File

@@ -209,12 +209,12 @@ class CreateFlowUtils {
}
fun toDisabledProviderList(
providerDataList: List<DisabledProviderData>,
providerDataList: List<DisabledProviderData>?,
context: Context,
): List<com.android.credentialmanager.createflow.DisabledProviderInfo> {
): List<com.android.credentialmanager.createflow.DisabledProviderInfo>? {
// TODO: get from the actual service info
val packageManager = context.packageManager
return providerDataList.map {
return providerDataList?.map {
val pkgInfo = packageManager
.getPackageInfo(it.providerFlattenedComponentName,
PackageManager.PackageInfoFlags.of(0))