Merge "Remove BAL feature flag" into udc-dev

This commit is contained in:
Nan Wu
2023-05-10 16:53:59 +00:00
committed by Android (Google) Code Review
2 changed files with 1 additions and 18 deletions

View File

@@ -46,7 +46,6 @@ import android.os.RemoteCallbackList;
import android.os.RemoteException; import android.os.RemoteException;
import android.os.TransactionTooLargeException; import android.os.TransactionTooLargeException;
import android.os.UserHandle; import android.os.UserHandle;
import android.provider.DeviceConfig;
import android.util.ArrayMap; import android.util.ArrayMap;
import android.util.ArraySet; import android.util.ArraySet;
import android.util.Slog; import android.util.Slog;
@@ -68,9 +67,6 @@ public final class PendingIntentRecord extends IIntentSender.Stub {
@EnabledAfter(targetSdkVersion = Build.VERSION_CODES.TIRAMISU) @EnabledAfter(targetSdkVersion = Build.VERSION_CODES.TIRAMISU)
@Overridable @Overridable
private static final long DEFAULT_RESCIND_BAL_PRIVILEGES_FROM_PENDING_INTENT_SENDER = 244637991; private static final long DEFAULT_RESCIND_BAL_PRIVILEGES_FROM_PENDING_INTENT_SENDER = 244637991;
private static final String ENABLE_DEFAULT_RESCIND_BAL_PRIVILEGES_FROM_PENDING_INTENT_SENDER =
"DefaultRescindBalPrivilegesFromPendingIntentSender__"
+ "enable_default_rescind_bal_privileges_from_pending_intent_sender";
public static final int FLAG_ACTIVITY_SENDER = 1 << 0; public static final int FLAG_ACTIVITY_SENDER = 1 << 0;
public static final int FLAG_BROADCAST_SENDER = 1 << 1; public static final int FLAG_BROADCAST_SENDER = 1 << 1;
@@ -373,13 +369,6 @@ public final class PendingIntentRecord extends IIntentSender.Stub {
: BackgroundStartPrivileges.NONE; : BackgroundStartPrivileges.NONE;
} }
private static boolean isDefaultRescindBalPrivilegesFromPendingIntentSenderEnabled() {
return DeviceConfig.getBoolean(
DeviceConfig.NAMESPACE_WINDOW_MANAGER,
ENABLE_DEFAULT_RESCIND_BAL_PRIVILEGES_FROM_PENDING_INTENT_SENDER,
false); // assume false if the property is unknown
}
/** /**
* Default {@link BackgroundStartPrivileges} to be used if the intent sender has not made an * Default {@link BackgroundStartPrivileges} to be used if the intent sender has not made an
* explicit choice. * explicit choice.
@@ -393,10 +382,9 @@ public final class PendingIntentRecord extends IIntentSender.Stub {
}) })
public static BackgroundStartPrivileges getDefaultBackgroundStartPrivileges( public static BackgroundStartPrivileges getDefaultBackgroundStartPrivileges(
int callingUid) { int callingUid) {
boolean isFlagEnabled = isDefaultRescindBalPrivilegesFromPendingIntentSenderEnabled();
boolean isChangeEnabledForApp = CompatChanges.isChangeEnabled( boolean isChangeEnabledForApp = CompatChanges.isChangeEnabled(
DEFAULT_RESCIND_BAL_PRIVILEGES_FROM_PENDING_INTENT_SENDER, callingUid); DEFAULT_RESCIND_BAL_PRIVILEGES_FROM_PENDING_INTENT_SENDER, callingUid);
if (isFlagEnabled && isChangeEnabledForApp) { if (isChangeEnabledForApp) {
return BackgroundStartPrivileges.ALLOW_FGS; return BackgroundStartPrivileges.ALLOW_FGS;
} else { } else {
return BackgroundStartPrivileges.ALLOW_BAL; return BackgroundStartPrivileges.ALLOW_BAL;

View File

@@ -144,9 +144,6 @@ import java.util.Set;
@RunWith(WindowTestRunner.class) @RunWith(WindowTestRunner.class)
public class ActivityStarterTests extends WindowTestsBase { public class ActivityStarterTests extends WindowTestsBase {
private static final String ENABLE_DEFAULT_RESCIND_BAL_PRIVILEGES_FROM_PENDING_INTENT_SENDER =
"DefaultRescindBalPrivilegesFromPendingIntentSender__"
+ "enable_default_rescind_bal_privileges_from_pending_intent_sender";
private static final int PRECONDITION_NO_CALLER_APP = 1; private static final int PRECONDITION_NO_CALLER_APP = 1;
private static final int PRECONDITION_NO_INTENT_COMPONENT = 1 << 1; private static final int PRECONDITION_NO_INTENT_COMPONENT = 1 << 1;
private static final int PRECONDITION_NO_ACTIVITY_INFO = 1 << 2; private static final int PRECONDITION_NO_ACTIVITY_INFO = 1 << 2;
@@ -184,8 +181,6 @@ public class ActivityStarterTests extends WindowTestsBase {
doReturn(AppOpsManager.MODE_DEFAULT).when(mAppOpsManager).checkOpNoThrow( doReturn(AppOpsManager.MODE_DEFAULT).when(mAppOpsManager).checkOpNoThrow(
eq(AppOpsManager.OP_SYSTEM_EXEMPT_FROM_ACTIVITY_BG_START_RESTRICTION), eq(AppOpsManager.OP_SYSTEM_EXEMPT_FROM_ACTIVITY_BG_START_RESTRICTION),
anyInt(), any()); anyInt(), any());
mDeviceConfig.set(ENABLE_DEFAULT_RESCIND_BAL_PRIVILEGES_FROM_PENDING_INTENT_SENDER,
String.valueOf(true));
} }
@After @After