Set adminExtras as an empty persistable bundle if left null in builder.

Bug: 235282962

Test: atest android.devicepolicy.gts.ManagedProvisioningRequiredAppsTest#newlyProvisionedManagedProfile_hasRequiredApps
Test: atest
android.devicepolicy.gts.ManagedProvisioningRequiredAppsTest#newlyProvisionedFullyManagedDevice_hasRequiredApps

Change-Id: I7242d370a85e9e74bdc5792809c57aa9db9de7a7
This commit is contained in:
Elis Elliott
2022-06-10 08:37:23 +00:00
parent b930850863
commit 498fb25fcd
2 changed files with 4 additions and 2 deletions

View File

@@ -301,6 +301,7 @@ public final class FullyManagedDeviceProvisioningParams implements Parcelable {
* Sets a {@link PersistableBundle} that contains admin-specific extras.
*/
@NonNull
//TODO(b/235783053) The adminExtras parameter is actually @Nullable.
public Builder setAdminExtras(@NonNull PersistableBundle adminExtras) {
mAdminExtras = adminExtras != null
? new PersistableBundle(adminExtras)
@@ -333,7 +334,7 @@ public final class FullyManagedDeviceProvisioningParams implements Parcelable {
mLocalTime,
mLocale,
mDeviceOwnerCanGrantSensorsPermissions,
mAdminExtras,
mAdminExtras != null ? mAdminExtras : new PersistableBundle(),
mDemoDevice);
}
}

View File

@@ -252,6 +252,7 @@ public final class ManagedProfileProvisioningParams implements Parcelable {
* Sets a {@link Bundle} that contains admin-specific extras.
*/
@NonNull
//TODO(b/235783053) The adminExtras parameter is actually @Nullable.
public Builder setAdminExtras(@NonNull PersistableBundle adminExtras) {
mAdminExtras = adminExtras != null
? new PersistableBundle(adminExtras)
@@ -274,7 +275,7 @@ public final class ManagedProfileProvisioningParams implements Parcelable {
mLeaveAllSystemAppsEnabled,
mOrganizationOwnedProvisioning,
mKeepingAccountOnMigration,
mAdminExtras);
mAdminExtras != null ? mAdminExtras : new PersistableBundle());
}
}