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

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

Issue 1363723002: [Cronet] Create Builders, rename UrlRequestContext to CronetEngine (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update Ben's tests Created 5 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/urlconnection/CronetURLStreamHandlerFactory.java
diff --git a/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetURLStreamHandlerFactory.java b/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetURLStreamHandlerFactory.java
index 2363a8f9ddde822f3d08f16d1076b6660432ac74..aadcd7b1f0c0060dccc8fb1a84f8cbc82c57504e 100644
--- a/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetURLStreamHandlerFactory.java
+++ b/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetURLStreamHandlerFactory.java
@@ -4,10 +4,7 @@
package org.chromium.net.urlconnection;
-import android.content.Context;
-
-import org.chromium.net.UrlRequestContext;
-import org.chromium.net.UrlRequestContextConfig;
+import org.chromium.net.CronetEngine;
import java.net.URLStreamHandler;
import java.net.URLStreamHandlerFactory;
@@ -42,21 +39,19 @@ import java.net.URLStreamHandlerFactory;
*/
public class CronetURLStreamHandlerFactory
implements URLStreamHandlerFactory {
- private final UrlRequestContext mRequestContext;
+ private final CronetEngine mCronetEngine;
/**
* Creates a {@link CronetURLStreamHandlerFactory} to handle HTTP and HTTPS
* traffic.
- * @param context application context.
- * @param config the configuration to be used.
+ * @param cronetEngine the {@link CronetEngine} to be used.
* @throws NullPointerException if config is null.
*/
- public CronetURLStreamHandlerFactory(Context context,
- UrlRequestContextConfig config) {
- if (config == null) {
- throw new NullPointerException("UrlRequestContextConfig is null.");
+ public CronetURLStreamHandlerFactory(CronetEngine cronetEngine) {
+ if (cronetEngine == null) {
+ throw new NullPointerException("CronetEngine is null.");
}
- mRequestContext = UrlRequestContext.createContext(context, config);
+ mCronetEngine = cronetEngine;
}
/**
@@ -66,7 +61,7 @@ public class CronetURLStreamHandlerFactory
@Override
public URLStreamHandler createURLStreamHandler(String protocol) {
if ("http".equals(protocol) || "https".equals(protocol)) {
- return new CronetHttpURLStreamHandler(mRequestContext);
+ return new CronetHttpURLStreamHandler(mCronetEngine);
}
return null;
}

Powered by Google App Engine
This is Rietveld 408576698