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

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

Issue 2069303002: Add new Cronet exception class for QUIC errors (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: header Created 4 years, 6 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/CronetBidirectionalStream.java
diff --git a/components/cronet/android/java/src/org/chromium/net/CronetBidirectionalStream.java b/components/cronet/android/java/src/org/chromium/net/CronetBidirectionalStream.java
index 5e943043e1d490592c137c0a7cccb463a0ff1b4c..0422df300f2de159c04c34e156ffeb993b97abed 100644
--- a/components/cronet/android/java/src/org/chromium/net/CronetBidirectionalStream.java
+++ b/components/cronet/android/java/src/org/chromium/net/CronetBidirectionalStream.java
@@ -555,13 +555,18 @@ class CronetBidirectionalStream extends BidirectionalStream {
@SuppressWarnings("unused")
@CalledByNative
- private void onError(
- int errorCode, int nativeError, String errorString, long receivedBytesCount) {
+ private void onError(int errorCode, int nativeError, int nativeQuicError, String errorString,
+ long receivedBytesCount) {
if (mResponseInfo != null) {
mResponseInfo.setReceivedBytesCount(receivedBytesCount);
}
- failWithException(new CronetException(
- "Exception in BidirectionalStream: " + errorString, errorCode, nativeError));
+ if (errorCode == UrlRequestException.ERROR_QUIC_PROTOCOL_FAILED) {
+ failWithException(new QuicException("Exception in BidirectionalStream: " + errorString,
+ nativeError, nativeQuicError));
+ } else {
+ failWithException(new CronetException(
+ "Exception in BidirectionalStream: " + errorString, errorCode, nativeError));
+ }
}
/**

Powered by Google App Engine
This is Rietveld 408576698