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

Commit 5047bef2 authored by ELIYAZ MOMIN (xWF)'s avatar ELIYAZ MOMIN (xWF)
Browse files

Revert "replace mContext.pm() refs w/ context.createContextAsUser.getPm()" 

This reverts commit cfbc6bc0.

Reason for revert: <Potential culprit for b/421510447  verifying through ABTD before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted>

Bug: 421510447,421499946
Change-Id: I75976f602ced57a568613480f3e48967e4f7120d
parent cfbc6bc0

src/com/android/server/telecom/CallScreeningServiceHelper.java

+4 −15
Original line number Diff line number Diff line
@@ -21,7 +21,6 @@ import android.content.ComponentName; 
import android.content.Context;

import android.content.Intent;

import android.content.ServiceConnection;

import android.content.pm.PackageManager;

import android.content.pm.ResolveInfo;

import android.os.Handler;

import android.os.IBinder;
@@ -32,7 +31,6 @@ import android.telecom.Log; 
import android.telecom.Logging.Session;

import android.text.TextUtils;



import com.android.internal.hidden_from_bootclasspath.com.android.server.telecom.flags.Flags;

import com.android.internal.telecom.ICallScreeningAdapter;

import com.android.internal.telecom.ICallScreeningService;

import com.android.server.telecom.flags.FeatureFlags;
@@ -168,8 +166,7 @@ public class CallScreeningServiceHelper { 
            }

        };



        if (!bindCallScreeningService(mContext, mUserHandle, mPackageName, serviceConnection,

                mFeatureFlags)) {

        if (!bindCallScreeningService(mContext, mUserHandle, mPackageName, serviceConnection)) {

            Log.i(this, "bindAndGetCallIdentification - bind failed");

            mFuture.complete(null);

        }
@@ -206,7 +203,7 @@ public class CallScreeningServiceHelper { 
     * @return {@code true} if binding succeeds, {@code false} otherwise.

     */

    public static boolean bindCallScreeningService(Context context, UserHandle userHandle,

            String packageName, ServiceConnection serviceConnection, FeatureFlags flags) {

            String packageName, ServiceConnection serviceConnection) {

        if (TextUtils.isEmpty(packageName)) {

            Log.i(TAG, "PackageName is empty. Not performing call screening.");

            return false;
@@ -214,16 +211,8 @@ public class CallScreeningServiceHelper { 


        Intent intent = new Intent(CallScreeningService.SERVICE_INTERFACE)

                .setPackage(packageName);



        List<ResolveInfo> entries;

        if (flags.resolveHiddenDependenciesTwo()) {

            entries = UserUtil.getPackageManagerFromUserHandler(context,

                    userHandle).queryIntentServicesAsUser(intent, 0, userHandle.getIdentifier());

        } else {

            entries = context.getPackageManager().queryIntentServicesAsUser(

        List<ResolveInfo> entries = context.getPackageManager().queryIntentServicesAsUser(

                intent, 0, userHandle.getIdentifier());

        }



        if (entries.isEmpty()) {

            Log.i(TAG, packageName + " has no call screening service defined.");

            return false;

src/com/android/server/telecom/InCallController.java

+10 −31
Original line number Diff line number Diff line
@@ -1380,12 +1380,7 @@ public class InCallController extends CallsManagerListenerBase implements 


                int uid;

                try {

                    if (mFeatureFlags.resolveHiddenDependenciesTwo()) {

                        uid = UserUtil.getPackageManagerFromUserHandler(mContext,

                                user).getPackageUidAsUser(pkg, user.getIdentifier());

                    } else {

                    uid = pkgManager.getPackageUidAsUser(pkg, user.getIdentifier());

                    }

                } catch (PackageManager.NameNotFoundException e) {

                    continue;

                }
@@ -2446,19 +2441,11 @@ public class InCallController extends CallsManagerListenerBase implements 
        PackageManager userPackageManager = userContext != null ?

                userContext.getPackageManager() : packageManager;



        List<ResolveInfo> entries;

        if (mFeatureFlags.resolveHiddenDependenciesTwo()) {

            entries = userPackageManager.queryIntentServicesAsUser(

                    serviceIntent,

                    PackageManager.GET_META_DATA | PackageManager.MATCH_DISABLED_COMPONENTS,

                    userHandle.getIdentifier());

        } else {

            entries = packageManager.queryIntentServicesAsUser(



        for (ResolveInfo entry : packageManager.queryIntentServicesAsUser(

                serviceIntent,

                PackageManager.GET_META_DATA | PackageManager.MATCH_DISABLED_COMPONENTS,

                    userHandle.getIdentifier());

        }

        for (ResolveInfo entry : entries) {

                userHandle.getIdentifier())) {

            ServiceInfo serviceInfo = entry.serviceInfo;



            if (serviceInfo != null) {
@@ -3026,17 +3013,9 @@ public class InCallController extends CallsManagerListenerBase implements 


        Intent intent = new Intent(InCallService.SERVICE_INTERFACE)

            .setPackage(ringingPackage);

        List<ResolveInfo> entries;

        if (mFeatureFlags.resolveHiddenDependenciesTwo()) {

            entries = UserUtil.getPackageManagerFromUserHandler(mContext,

                    userHandle).queryIntentServicesAsUser(

        List<ResolveInfo> entries = mContext.getPackageManager().queryIntentServicesAsUser(

                intent, PackageManager.GET_META_DATA,

                userHandle.getIdentifier());

        } else {

            entries = mContext.getPackageManager().queryIntentServicesAsUser(

                    intent, PackageManager.GET_META_DATA,

                    userHandle.getIdentifier());

        }

        if (entries.isEmpty()) {

            Log.w(this, "doesConnectedDialerSupportRinging: couldn't find dialer's package info"

                    + " <sad trombone>");

src/com/android/server/telecom/PhoneAccountRegistrar.java

+1 −6
Original line number Diff line number Diff line
@@ -801,12 +801,7 @@ public class PhoneAccountRegistrar { 


    private List<ResolveInfo> resolveComponent(ComponentName componentName,

            UserHandle userHandle) {

        PackageManager pm;

        if (mTelecomFeatureFlags.resolveHiddenDependenciesTwo()) {

            pm = UserUtil.getPackageManagerFromUserHandler(mContext, userHandle);

        } else {

            pm = mContext.getPackageManager();

        }

        PackageManager pm = mContext.getPackageManager();

        Intent intent = new Intent(ConnectionService.SERVICE_INTERFACE);

        intent.setComponent(componentName);

        try {

src/com/android/server/telecom/TelecomBroadcastIntentProcessor.java

+4 −18
Original line number Diff line number Diff line
@@ -26,11 +26,9 @@ import android.os.UserHandle; 
import android.telecom.Log;

import android.widget.Toast;



import com.android.server.telecom.flags.FeatureFlags;

import com.android.server.telecom.ui.ConfirmCallDialogActivity;

import com.android.server.telecom.ui.DisconnectedCallNotifier;



import java.util.ArrayList;

import java.util.List;



public final class TelecomBroadcastIntentProcessor {
@@ -115,15 +113,10 @@ public final class TelecomBroadcastIntentProcessor { 


    private final Context mContext;

    private final CallsManager mCallsManager;

    private final FeatureFlags mFeatureFlags;



    public TelecomBroadcastIntentProcessor(

            Context context,

            CallsManager callsManager,

            FeatureFlags flags) {

    public TelecomBroadcastIntentProcessor(Context context, CallsManager callsManager) {

        mContext = context;

        mCallsManager = callsManager;

        mFeatureFlags = flags;

    }



    public void processIntent(Intent intent) {
@@ -294,16 +287,9 @@ public final class TelecomBroadcastIntentProcessor { 
    private void sendSmsIntent(Intent intent, UserHandle userHandle) {

        Intent callIntent = new Intent(Intent.ACTION_SENDTO, intent.getData());

        callIntent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK);

        List<ResolveInfo> activities;

        if (mFeatureFlags.resolveHiddenDependenciesTwo()) {

            activities = UserUtil.getPackageManagerFromUserHandler(mContext,

                    userHandle).queryIntentActivitiesAsUser(callIntent,

                    PackageManager.MATCH_DEFAULT_ONLY, userHandle.getIdentifier());

        } else {

        PackageManager packageManager = mContext.getPackageManager();

            activities = packageManager.queryIntentActivitiesAsUser(

        List<ResolveInfo> activities = packageManager.queryIntentActivitiesAsUser(

                callIntent, PackageManager.MATCH_DEFAULT_ONLY, userHandle.getIdentifier());

        }

        if (activities.size() > 0) {

            mContext.startActivityAsUser(callIntent, userHandle);

        } else {

src/com/android/server/telecom/TelecomSystem.java

+3 −10
Original line number Diff line number Diff line
@@ -24,7 +24,6 @@ import android.content.Context; 
import android.content.Intent;

import android.content.IntentFilter;

import android.content.ServiceConnection;

import android.content.pm.PackageManager;

import android.content.pm.ResolveInfo;

import android.net.Uri;

import android.os.BugreportManager;
@@ -321,14 +320,8 @@ public class TelecomSystem { 
                                @Override

                                public List<ResolveInfo> queryIntentServicesAsUser(

                                        @NonNull Intent intent, int flags, int userId) {

                                    PackageManager pm;

                                    if (mFeatureFlags.resolveHiddenDependenciesTwo()) {

                                        pm = UserUtil.getPackageManagerFromUserHandler(mContext,

                                                UserHandle.of(userId));

                                    } else {

                                        pm = mContext.getPackageManager();

                                    }

                                    return pm.queryIntentServicesAsUser(intent, flags, userId);

                                    return mContext.getPackageManager().queryIntentServicesAsUser(

                                            intent, flags, userId);

                                }



                                @Override
@@ -506,7 +499,7 @@ public class TelecomSystem { 
            mCallIntentProcessor = new CallIntentProcessor(mContext, mCallsManager,

                    defaultDialerCache, featureFlags);

            mTelecomBroadcastIntentProcessor = new TelecomBroadcastIntentProcessor(

                    mContext, mCallsManager, mFeatureFlags);

                    mContext, mCallsManager);



            // Register the receiver for the dialer secret codes, used to enable extended logging.

            mDialerCodeReceiver = new DialerCodeReceiver(mCallsManager);