Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1159)

Unified Diff: components/cronet/android/java/src/org/chromium/net/CronetUrlRequest.java

Issue 1492583002: Add HttpUrlConnection backed implementation of CronetEngine. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix tests Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/cronet/android/java/src/org/chromium/net/CronetUrlRequest.java
diff --git a/components/cronet/android/java/src/org/chromium/net/CronetUrlRequest.java b/components/cronet/android/java/src/org/chromium/net/CronetUrlRequest.java
index 6ec5b8daa18d8cb62100aeb438ee940b207c9f3f..eca3aabed1b4b02b8bc396f3421106b5bbe490e5 100644
--- a/components/cronet/android/java/src/org/chromium/net/CronetUrlRequest.java
+++ b/components/cronet/android/java/src/org/chromium/net/CronetUrlRequest.java
@@ -266,11 +266,9 @@ final class CronetUrlRequest implements UrlRequest {
@Override
public void readNew(ByteBuffer buffer) {
+ Preconditions.checkHasRemaining(buffer);
+ Preconditions.checkDirect(buffer);
synchronized (mUrlRequestAdapterLock) {
- if (!buffer.hasRemaining()) {
- throw new IllegalArgumentException("ByteBuffer is already full.");
- }
-
if (!mWaitingOnRead) {
throw new IllegalStateException("Unexpected read attempt.");
}
@@ -287,7 +285,7 @@ final class CronetUrlRequest implements UrlRequest {
mWaitingOnRead = true;
// Since accessing byteBuffer's memory failed, it's presumably
// not a direct ByteBuffer.
- throw new IllegalArgumentException("byteBuffer must be a direct ByteBuffer.");
+ throw new IllegalArgumentException("Unable to call native read");
}
}
}

Powered by Google App Engine
This is Rietveld 408576698