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

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

Issue 2839413003: Delete Cronet Data Reduction Proxy integration (Closed)
Patch Set: Created 3 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/impl/CronetEngineBuilderImpl.java
diff --git a/components/cronet/android/java/src/org/chromium/net/impl/CronetEngineBuilderImpl.java b/components/cronet/android/java/src/org/chromium/net/impl/CronetEngineBuilderImpl.java
index 842d11ca7473230be35b847c565def69967dbc0b..813db69beacbecebce23fa9b37d66141ef557c83 100644
--- a/components/cronet/android/java/src/org/chromium/net/impl/CronetEngineBuilderImpl.java
+++ b/components/cronet/android/java/src/org/chromium/net/impl/CronetEngineBuilderImpl.java
@@ -84,10 +84,6 @@ public abstract class CronetEngineBuilderImpl extends ICronetEngineBuilder {
private boolean mHttp2Enabled;
private boolean mSdchEnabled;
private boolean mBrotiEnabled;
- private String mDataReductionProxyKey;
- private String mDataReductionProxyPrimaryProxy;
- private String mDataReductionProxyFallbackProxy;
- private String mDataReductionProxySecureProxyCheckUrl;
private boolean mDisableCache;
private int mHttpCacheMode;
private long mHttpCacheMaxSize;
@@ -199,41 +195,6 @@ public abstract class CronetEngineBuilderImpl extends ICronetEngineBuilder {
return mBrotiEnabled;
}
- @Override
- public CronetEngineBuilderImpl enableDataReductionProxy(String key) {
- mDataReductionProxyKey = key;
- return this;
- }
-
- String dataReductionProxyKey() {
- return mDataReductionProxyKey;
- }
-
- @Override
- public CronetEngineBuilderImpl setDataReductionProxyOptions(
- String primaryProxy, String fallbackProxy, String secureProxyCheckUrl) {
- if (primaryProxy.isEmpty() || fallbackProxy.isEmpty() || secureProxyCheckUrl.isEmpty()) {
- throw new IllegalArgumentException(
- "Primary and fallback proxies and check url must be set");
- }
- mDataReductionProxyPrimaryProxy = primaryProxy;
- mDataReductionProxyFallbackProxy = fallbackProxy;
- mDataReductionProxySecureProxyCheckUrl = secureProxyCheckUrl;
- return this;
- }
-
- String dataReductionProxyPrimaryProxy() {
- return mDataReductionProxyPrimaryProxy;
- }
-
- String dataReductionProxyFallbackProxy() {
- return mDataReductionProxyFallbackProxy;
- }
-
- String dataReductionProxySecureProxyCheckUrl() {
- return mDataReductionProxySecureProxyCheckUrl;
- }
-
@IntDef({
HTTP_CACHE_DISABLED, HTTP_CACHE_IN_MEMORY, HTTP_CACHE_DISK_NO_HTTP, HTTP_CACHE_DISK,
})

Powered by Google App Engine
This is Rietveld 408576698