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

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

Issue 2069303002: Add new Cronet exception class for QUIC errors (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Switch to using NetError 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/api/src/org/chromium/net/UrlRequestException.java
diff --git a/components/cronet/android/api/src/org/chromium/net/UrlRequestException.java b/components/cronet/android/api/src/org/chromium/net/UrlRequestException.java
index 92201aa32471128ce721a81cab80fbf8b061f73b..f1ab73877501e30d4e8213924be7808926c9da08 100644
--- a/components/cronet/android/api/src/org/chromium/net/UrlRequestException.java
+++ b/components/cronet/android/api/src/org/chromium/net/UrlRequestException.java
@@ -14,7 +14,9 @@ import java.io.IOException;
* {@link #getErrorCode} will return {@link #ERROR_LISTENER_EXCEPTION_THROWN}.
* <li>Cronet fails to process a network request. In this case
* {@link #getErrorCode} and {@link #getCronetInternalErrorCode} can be used to get more
- * information about the specific type of failure.
+ * information about the specific type of failure. If {@link #getCronetInternalErrorCode}
+ * returns QUIC_PROTOCOL_ERROR, this exception can be cast to a {@link QuicException}
pauljensen 2016/06/30 17:35:12 It's quite awkward to reference a value here witho
mgersh 2016/06/30 22:55:53 Done. (Was the different name just to avoid saying
+ * which can provide further details.
* </ul>
*/
public class UrlRequestException extends IOException {

Powered by Google App Engine
This is Rietveld 408576698