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

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

Issue 2339223002: Cronet API Refactoring (Closed)
Patch Set: Rebase & Conflict Resolution Created 4 years, 2 months 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/impl/CronetBidirectionalStream.java
diff --git a/components/cronet/android/java/src/org/chromium/net/impl/CronetBidirectionalStream.java b/components/cronet/android/java/src/org/chromium/net/impl/CronetBidirectionalStream.java
index 9a1fb8b9c0ec157daf03871baa5f805fcd28437d..1d007642b67f6f79932f6c2688b2d375904c1323 100644
--- a/components/cronet/android/java/src/org/chromium/net/impl/CronetBidirectionalStream.java
+++ b/components/cronet/android/java/src/org/chromium/net/impl/CronetBidirectionalStream.java
@@ -11,7 +11,7 @@ import org.chromium.base.annotations.JNINamespace;
import org.chromium.base.annotations.NativeClassQualifiedName;
import org.chromium.net.BidirectionalStream;
import org.chromium.net.CronetException;
-import org.chromium.net.Preconditions;
+import org.chromium.net.ExperimentalBidirectionalStream;
import org.chromium.net.QuicException;
import org.chromium.net.RequestFinishedInfo;
import org.chromium.net.RequestPriority;
@@ -39,7 +39,7 @@ import javax.annotation.concurrent.GuardedBy;
*/
@JNINamespace("cronet")
@VisibleForTesting
-public class CronetBidirectionalStream extends BidirectionalStream {
+public class CronetBidirectionalStream extends ExperimentalBidirectionalStream {
/**
* States of BidirectionalStream are tracked in mReadState and mWriteState.
* The write state is separated out as it changes independently of the read state.
@@ -224,8 +224,8 @@ public class CronetBidirectionalStream extends BidirectionalStream {
}
CronetBidirectionalStream(CronetUrlRequestContext requestContext, String url,
- @BidirectionalStream.Builder.StreamPriority int priority, Callback callback,
- Executor executor, String httpMethod, List<Map.Entry<String, String>> requestHeaders,
+ @CronetEngineBase.StreamPriority int priority, Callback callback, Executor executor,
+ String httpMethod, List<Map.Entry<String, String>> requestHeaders,
boolean delayRequestHeadersUntilNextFlush, Collection<Object> requestAnnotations) {
mRequestContext = requestContext;
mInitialUrl = url;
@@ -684,8 +684,7 @@ public class CronetBidirectionalStream extends BidirectionalStream {
return headersArray;
}
- private static int convertStreamPriority(
- @BidirectionalStream.Builder.StreamPriority int priority) {
+ private static int convertStreamPriority(@CronetEngineBase.StreamPriority int priority) {
switch (priority) {
case Builder.STREAM_PRIORITY_IDLE:
return RequestPriority.IDLE;

Powered by Google App Engine
This is Rietveld 408576698