Merge "Set adminExtras as an empty persistable bundle if left null in builder." into tm-dev

This commit is contained in:
TreeHugger Robot
2022-06-17 10:49:38 +00:00
committed by Android (Google) Code Review
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. * Sets a {@link PersistableBundle} that contains admin-specific extras.
*/ */
@NonNull @NonNull
//TODO(b/235783053) The adminExtras parameter is actually @Nullable.
public Builder setAdminExtras(@NonNull PersistableBundle adminExtras) { public Builder setAdminExtras(@NonNull PersistableBundle adminExtras) {
mAdminExtras = adminExtras != null mAdminExtras = adminExtras != null
? new PersistableBundle(adminExtras) ? new PersistableBundle(adminExtras)
@@ -333,7 +334,7 @@ public final class FullyManagedDeviceProvisioningParams implements Parcelable {
mLocalTime, mLocalTime,
mLocale, mLocale,
mDeviceOwnerCanGrantSensorsPermissions, mDeviceOwnerCanGrantSensorsPermissions,
mAdminExtras, mAdminExtras != null ? mAdminExtras : new PersistableBundle(),
mDemoDevice); mDemoDevice);
} }
} }

View File

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