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

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

Issue 1363723002: [Cronet] Create Builders, rename UrlRequestContext to CronetEngine (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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/HttpUrlRequestFactory.java
diff --git a/components/cronet/android/java/src/org/chromium/net/HttpUrlRequestFactory.java b/components/cronet/android/java/src/org/chromium/net/HttpUrlRequestFactory.java
index fe84f5fae897ac33259b5478962f3bb3944317c0..c4cfe1c3ec2bf2ee435b93a93b6c3b8e9bf1a4d0 100644
--- a/components/cronet/android/java/src/org/chromium/net/HttpUrlRequestFactory.java
+++ b/components/cronet/android/java/src/org/chromium/net/HttpUrlRequestFactory.java
@@ -14,7 +14,7 @@ import java.util.Map;
/**
* A factory for {@link HttpUrlRequest}'s, which uses the best HTTP stack
* available on the current platform.
- * @deprecated Use {@link UrlRequestContext} instead.
+ * @deprecated Use {@link CronetEngine} instead.
*/
@Deprecated
public abstract class HttpUrlRequestFactory {
@@ -24,14 +24,14 @@ public abstract class HttpUrlRequestFactory {
"org.chromium.net.ChromiumUrlRequestFactory";
public static HttpUrlRequestFactory createFactory(
- Context context, UrlRequestContextConfig config) {
+ Context context, CronetEngine.Builder builder) {
HttpUrlRequestFactory factory = null;
- if (!config.legacyMode()) {
- factory = createChromiumFactory(context, config);
+ if (!builder.legacyMode()) {
+ factory = createChromiumFactory(context, builder);
}
if (factory == null) {
// Default to HttpUrlConnection-based networking.
- factory = new HttpUrlConnectionUrlRequestFactory(context, config);
+ factory = new HttpUrlConnectionUrlRequestFactory(context, builder);
}
Log.i(TAG, "Using network stack: " + factory.getName());
return factory;
@@ -80,7 +80,7 @@ public abstract class HttpUrlRequestFactory {
public abstract void stopNetLog();
private static HttpUrlRequestFactory createChromiumFactory(
- Context context, UrlRequestContextConfig config) {
+ Context context, CronetEngine.Builder builder) {
HttpUrlRequestFactory factory = null;
try {
Class<? extends HttpUrlRequestFactory> factoryClass =
@@ -88,10 +88,8 @@ public abstract class HttpUrlRequestFactory {
.loadClass(CHROMIUM_URL_REQUEST_FACTORY)
.asSubclass(HttpUrlRequestFactory.class);
Constructor<? extends HttpUrlRequestFactory> constructor =
- factoryClass.getConstructor(
- Context.class, UrlRequestContextConfig.class);
- HttpUrlRequestFactory chromiumFactory =
- constructor.newInstance(context, config);
+ factoryClass.getConstructor(Context.class, CronetEngine.Builder.class);
+ HttpUrlRequestFactory chromiumFactory = constructor.newInstance(context, builder);
if (chromiumFactory.isEnabled()) {
factory = chromiumFactory;
}

Powered by Google App Engine
This is Rietveld 408576698