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

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

Issue 937513003: Add Data Saver support to Cronet (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments Created 5 years, 8 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/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 cd7025757f8ca4a54c00d44ec2f62911de05b79f..4990e1edf55eb366f12cf57cf2d09f34814afa7d 100644
--- a/components/cronet/android/java/src/org/chromium/net/CronetUrlRequest.java
+++ b/components/cronet/android/java/src/org/chromium/net/CronetUrlRequest.java
@@ -108,17 +108,19 @@ final class CronetUrlRequest implements UrlRequest {
private final HeadersList mAllHeaders = new HeadersList();
private final boolean mWasCached;
private final String mNegotiatedProtocol;
+ private final String mProxyServer;
private Map<String, List<String>> mResponseHeaders;
private List<Pair<String, String>> mUnmodifiableAllHeaders;
NativeResponseInfo(String[] urlChain, int httpStatusCode,
String httpStatusText, boolean wasCached,
- String negotiatedProtocol) {
+ String negotiatedProtocol, String proxyServer) {
mResponseInfoUrlChain = urlChain;
mHttpStatusCode = httpStatusCode;
mHttpStatusText = httpStatusText;
mWasCached = wasCached;
mNegotiatedProtocol = negotiatedProtocol;
+ mProxyServer = proxyServer;
}
@Override
@@ -178,6 +180,11 @@ final class CronetUrlRequest implements UrlRequest {
public String getNegotiatedProtocol() {
return mNegotiatedProtocol;
}
+
+ @Override
+ public String getProxyServer() {
+ return mProxyServer;
+ }
};
static final class NativeExtendedResponseInfo implements
@@ -423,7 +430,8 @@ final class CronetUrlRequest implements UrlRequest {
httpStatusCode,
nativeGetHttpStatusText(urlRequestAdapter),
nativeGetWasCached(urlRequestAdapter),
- nativeGetNegotiatedProtocol(urlRequestAdapter));
+ nativeGetNegotiatedProtocol(urlRequestAdapter),
+ nativeGetProxyServer(urlRequestAdapter));
nativePopulateResponseHeaders(urlRequestAdapter,
responseInfo.mAllHeaders);
return responseInfo;
@@ -713,6 +721,9 @@ final class CronetUrlRequest implements UrlRequest {
private native String nativeGetNegotiatedProtocol(long nativePtr);
@NativeClassQualifiedName("CronetURLRequestAdapter")
+ private native String nativeGetProxyServer(long nativePtr);
+
+ @NativeClassQualifiedName("CronetURLRequestAdapter")
private native boolean nativeGetWasCached(long nativePtr);
@NativeClassQualifiedName("CronetURLRequestAdapter")

Powered by Google App Engine
This is Rietveld 408576698