diff --git a/res/xml/accounts_work_dashboard_settings_credman.xml b/res/xml/accounts_work_dashboard_settings_credman.xml index 15299208a4e..bb9976ec79c 100644 --- a/res/xml/accounts_work_dashboard_settings_credman.xml +++ b/res/xml/accounts_work_dashboard_settings_credman.xml @@ -28,8 +28,8 @@ android:title="@string/credman_chosen_app_title"> { - context.startActivityAsUser(addNewServiceIntent, UserHandle.of(mUserId)); + context.startActivityAsUser(addNewServiceIntent, UserHandle.of(getUser())); return true; }); preference.setTitle(R.string.print_menu_item_add_service); @@ -212,17 +213,17 @@ public class DefaultCombinedPicker extends DefaultAppPickerFragment { private List getAllProviders() { final Context context = getContext(); final List autofillProviders = - AutofillServiceInfo.getAvailableServices(context, mUserId); + AutofillServiceInfo.getAvailableServices(context, getUser()); final CredentialManager service = getCredentialProviderService(); final List credManProviders = new ArrayList<>(); if (service != null) { credManProviders.addAll( service.getCredentialProviderServices( - mUserId, CredentialManager.PROVIDER_FILTER_USER_PROVIDERS_ONLY)); + getUser(), CredentialManager.PROVIDER_FILTER_USER_PROVIDERS_ONLY)); } - final String selectedAutofillProvider = getSelectedAutofillProvider(context, mUserId); + final String selectedAutofillProvider = getSelectedAutofillProvider(context, getUser()); return CombinedProviderInfo.buildMergedList( autofillProviders, credManProviders, selectedAutofillProvider); } @@ -244,7 +245,7 @@ public class DefaultCombinedPicker extends DefaultAppPickerFragment { new DefaultAppInfo( context, mPm, - mUserId, + getUser(), cpi.getApplicationInfo(), cpi.getSettingsSubtitle(), true)); @@ -253,7 +254,7 @@ public class DefaultCombinedPicker extends DefaultAppPickerFragment { new DefaultAppInfo( context, mPm, - mUserId, + getUser(), brandingService, cpi.getSettingsSubtitle(), true)); @@ -350,7 +351,7 @@ public class DefaultCombinedPicker extends DefaultAppPickerFragment { } Settings.Secure.putStringForUser( - getContext().getContentResolver(), AUTOFILL_SETTING, autofillProvider, mUserId); + getContext().getContentResolver(), AUTOFILL_SETTING, autofillProvider, getUser()); final CredentialManager service = getCredentialProviderService(); if (service == null) { @@ -362,7 +363,7 @@ public class DefaultCombinedPicker extends DefaultAppPickerFragment { final List credManProviders = new ArrayList<>(); for (CredentialProviderInfo cpi : service.getCredentialProviderServices( - mUserId, CredentialManager.PROVIDER_FILTER_USER_PROVIDERS_ONLY)) { + getUser(), CredentialManager.PROVIDER_FILTER_USER_PROVIDERS_ONLY)) { if (cpi.isEnabled() && !cpi.isPrimary()) { credManProviders.add(cpi.getServiceInfo().getComponentName().flattenToString()); @@ -379,7 +380,7 @@ public class DefaultCombinedPicker extends DefaultAppPickerFragment { service.setEnabledProviders( primaryCredManProviders, credManProviders, - mUserId, + getUser(), ContextCompat.getMainExecutor(getContext()), new OutcomeReceiver() { @Override @@ -393,4 +394,11 @@ public class DefaultCombinedPicker extends DefaultAppPickerFragment { } }); } + + protected int getUser() { + if (mIntentSenderUserId >= 0) { + return mIntentSenderUserId; + } + return UserHandle.myUserId(); + } } diff --git a/src/com/android/settings/applications/credentials/DefaultCombinedPickerWork.java b/src/com/android/settings/applications/credentials/DefaultCombinedPickerWork.java new file mode 100644 index 00000000000..9808502b63e --- /dev/null +++ b/src/com/android/settings/applications/credentials/DefaultCombinedPickerWork.java @@ -0,0 +1,31 @@ +/* + * Copyright (C) 2023 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package com.android.settings.applications.credentials; + +import android.os.UserHandle; +import android.os.UserManager; + +import com.android.settings.Utils; + +public class DefaultCombinedPickerWork extends DefaultCombinedPicker { + + @Override + protected int getUser() { + UserHandle workProfile = Utils.getManagedProfile(UserManager.get(getContext())); + return workProfile.getIdentifier(); + } +} diff --git a/src/com/android/settings/applications/credentials/DefaultCombinedPreferenceController.java b/src/com/android/settings/applications/credentials/DefaultCombinedPreferenceController.java index 64d4f0dd5ed..34881ab6703 100644 --- a/src/com/android/settings/applications/credentials/DefaultCombinedPreferenceController.java +++ b/src/com/android/settings/applications/credentials/DefaultCombinedPreferenceController.java @@ -24,6 +24,7 @@ import android.content.pm.ResolveInfo; import android.content.pm.ServiceInfo; import android.credentials.CredentialManager; import android.credentials.CredentialProviderInfo; +import android.os.UserHandle; import android.provider.Settings; import android.service.autofill.AutofillService; import android.service.autofill.AutofillServiceInfo; @@ -75,13 +76,13 @@ public class DefaultCombinedPreferenceController extends DefaultAppPreferenceCon return null; } final AutofillSettingIntentProvider intentProvider = - new AutofillSettingIntentProvider(mContext, mUserId, info.getKey()); + new AutofillSettingIntentProvider(mContext, getUser(), info.getKey()); return intentProvider.getIntent(); } @Override protected DefaultAppInfo getDefaultAppInfo() { - List providers = getAllProviders(mUserId); + List providers = getAllProviders(getUser()); CombinedProviderInfo topProvider = CombinedProviderInfo.getTopProvider(providers); if (topProvider != null) { ServiceInfo brandingService = topProvider.getBrandingService(); @@ -89,7 +90,7 @@ public class DefaultCombinedPreferenceController extends DefaultAppPreferenceCon return new DefaultAppInfo( mContext, mPackageManager, - mUserId, + getUser(), topProvider.getApplicationInfo(), topProvider.getSettingsSubtitle(), true); @@ -97,7 +98,7 @@ public class DefaultCombinedPreferenceController extends DefaultAppPreferenceCon return new DefaultAppInfo( mContext, mPackageManager, - mUserId, + getUser(), brandingService, topProvider.getSettingsSubtitle(), true); @@ -178,4 +179,8 @@ public class DefaultCombinedPreferenceController extends DefaultAppPreferenceCon return null; } } + + protected int getUser() { + return UserHandle.myUserId(); + } } diff --git a/src/com/android/settings/applications/credentials/DefaultWorkCombinedPreferenceController.java b/src/com/android/settings/applications/credentials/DefaultWorkCombinedPreferenceController.java index 0990f3b2f9f..74bf9026b2f 100644 --- a/src/com/android/settings/applications/credentials/DefaultWorkCombinedPreferenceController.java +++ b/src/com/android/settings/applications/credentials/DefaultWorkCombinedPreferenceController.java @@ -44,7 +44,7 @@ public class DefaultWorkCombinedPreferenceController extends DefaultCombinedPref @Override public String getPreferenceKey() { - return "default_autofill_work"; + return "default_credman_autofill_main_work"; } @Override @@ -81,4 +81,9 @@ public class DefaultWorkCombinedPreferenceController extends DefaultCombinedPref protected void startActivity(Intent intent) { mContext.startActivityAsUser(intent, mUserHandle); } + + @Override + protected int getUser() { + return mUserHandle.getIdentifier(); + } }