Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b467333c authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Revert "Add a blacklist for preinstalled carrier apps."" into rvc-dev am: ddb640a8 

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

Change-Id: Icea3e45139b073ba4caee9fc399590b42cba4f01
parents 5674d0ac ddb640a8

core/res/res/values/config.xml

+0 −4
Original line number Diff line number Diff line
@@ -4313,10 +4313,6 @@ 
    notifications until they target R -->

    <string-array name="config_notificationMsgPkgsAllowedAsConvos" translatable="false"/>



    <!-- Contains a blacklist of apps that should not get pre-installed carrier app permission

         grants, even if the UICC claims that the app should be privileged. See b/138150105 -->

    <string-array name="config_restrictedPreinstalledCarrierApps" translatable="false"/>



    <!-- Sharesheet: define a max number of targets per application for new shortcuts-based direct share introduced in Q -->

    <integer name="config_maxShortcutTargetsPerApp">3</integer>

core/res/res/values/symbols.xml

+0 −1
Original line number Diff line number Diff line
@@ -3827,7 +3827,6 @@ 


  <java-symbol type="string" name="config_defaultSupervisionProfileOwnerComponent" />

  <java-symbol type="bool" name="config_inflateSignalStrength" />

  <java-symbol type="array" name="config_restrictedPreinstalledCarrierApps" />



  <java-symbol type="drawable" name="android_logotype" />

  <java-symbol type="layout" name="platlogo_layout" />

telephony/common/com/android/internal/telephony/CarrierAppUtils.java

+1 −7
Original line number Diff line number Diff line
@@ -21,7 +21,6 @@ import android.content.ContentResolver; 
import android.content.Context;

import android.content.pm.ApplicationInfo;

import android.content.pm.PackageManager;

import android.content.res.Resources;

import android.os.SystemConfigManager;

import android.os.UserHandle;

import android.permission.PermissionManager;
@@ -30,9 +29,7 @@ import android.telephony.TelephonyManager; 
import android.util.ArrayMap;

import android.util.Log;



import com.android.internal.R;

import com.android.internal.annotations.VisibleForTesting;

import com.android.internal.telephony.util.ArrayUtils;



import java.util.ArrayList;

import java.util.List;
@@ -162,12 +159,9 @@ public final class CarrierAppUtils { 
        try {

            for (ApplicationInfo ai : candidates) {

                String packageName = ai.packageName;

                String[] restrictedCarrierApps = Resources.getSystem().getStringArray(

                        R.array.config_restrictedPreinstalledCarrierApps);

                boolean hasPrivileges = telephonyManager != null

                        && telephonyManager.checkCarrierPrivilegesForPackageAnyPhone(packageName)

                                == TelephonyManager.CARRIER_PRIVILEGE_STATUS_HAS_ACCESS

                        && !ArrayUtils.contains(restrictedCarrierApps, packageName);

                                == TelephonyManager.CARRIER_PRIVILEGE_STATUS_HAS_ACCESS;



                // add hiddenUntilInstalled flag for carrier apps and associated apps

                packageManager.setSystemAppState(