Merge "Revert "Reject invalid input.""

This commit is contained in:
Kweku Adams
2021-06-25 21:09:47 +00:00
committed by Android (Google) Code Review
2 changed files with 1 additions and 14 deletions

View File

@@ -1250,11 +1250,6 @@ public class JobInfo implements Parcelable {
*/
public Builder setEstimatedNetworkBytes(@BytesLong long downloadBytes,
@BytesLong long uploadBytes) {
if ((downloadBytes != NETWORK_BYTES_UNKNOWN && downloadBytes < 0)
|| (uploadBytes != NETWORK_BYTES_UNKNOWN && uploadBytes < 0)) {
throw new IllegalArgumentException(
"Can't provide negative estimated network usage");
}
mNetworkDownloadBytes = downloadBytes;
mNetworkUploadBytes = uploadBytes;
return this;
@@ -1652,16 +1647,12 @@ public class JobInfo implements Parcelable {
/**
* @hide
*/
public final void enforceValidity() {
public void enforceValidity() {
// Check that network estimates require network type
if ((networkDownloadBytes > 0 || networkUploadBytes > 0) && networkRequest == null) {
throw new IllegalArgumentException(
"Can't provide estimated network usage without requiring a network");
}
if ((networkDownloadBytes != NETWORK_BYTES_UNKNOWN && networkDownloadBytes < 0)
|| (networkUploadBytes != NETWORK_BYTES_UNKNOWN && networkUploadBytes < 0)) {
throw new IllegalArgumentException("Can't provide negative estimated network usage");
}
// Check that a deadline was not set on a periodic job.
if (isPeriodic) {

View File

@@ -68,10 +68,6 @@ final public class JobWorkItem implements Parcelable {
* uploaded by this job work item, in bytes.
*/
public JobWorkItem(Intent intent, @BytesLong long downloadBytes, @BytesLong long uploadBytes) {
if ((downloadBytes != NETWORK_BYTES_UNKNOWN && downloadBytes < 0)
|| (uploadBytes != NETWORK_BYTES_UNKNOWN && uploadBytes < 0)) {
throw new IllegalArgumentException("Can't provide negative estimated network usage");
}
mIntent = intent;
mNetworkDownloadBytes = downloadBytes;
mNetworkUploadBytes = uploadBytes;