Chromium Code Reviews| Index: components/cronet/android/java/src/org/chromium/net/UrlRequestFactory.java |
| diff --git a/components/cronet/android/java/src/org/chromium/net/UrlRequestFactory.java b/components/cronet/android/java/src/org/chromium/net/UrlRequestFactory.java |
| index 2b08e7d99472e8a07a813d78446f6f608797ebb8..4a3bc18395c901b056ac7f79a90e92dfa7e858a1 100644 |
| --- a/components/cronet/android/java/src/org/chromium/net/UrlRequestFactory.java |
| +++ b/components/cronet/android/java/src/org/chromium/net/UrlRequestFactory.java |
| @@ -4,13 +4,21 @@ |
| package org.chromium.net; |
| +import android.content.Context; |
| +import android.util.Log; |
| + |
| +import java.lang.reflect.Constructor; |
| import java.util.concurrent.Executor; |
| /** |
| * A factory for {@link UrlRequest}'s, which uses the best HTTP stack |
| * available on the current platform. |
| */ |
| -public interface UrlRequestFactory { |
| +public abstract class UrlRequestFactory { |
| + private static final String TAG = "UrlRequestFactory"; |
| + private static final String CRONET_URL_REQUEST_CONTEXT = |
| + "org.chromium.net.CronetUrlRequestContext"; |
| + |
| /** |
| * Creates an UrlRequest object. All UrlRequest functions must |
| * be called on the Executor's thread, and all callbacks will be called |
| @@ -23,4 +31,58 @@ public interface UrlRequestFactory { |
| */ |
| public abstract UrlRequest createRequest(String url, |
| UrlRequestListener listener, Executor executor); |
| + |
| + public static UrlRequestFactory createFactory(Context context, |
| + String userAgent, HttpUrlRequestFactoryConfig config) { |
| + UrlRequestFactory factory = null; |
| + if (config.userAgent().isEmpty()) |
| + config.setUserAgent(UserAgent.from(context)); |
| + |
| + if (!config.legacyMode()) { |
| + factory = createCronetFactory(context, config); |
| + } |
| + if (factory == null) { |
| + // TODO(mef): Fallback to stub implementation. |
| + factory = createCronetFactory(context, config); |
| + } |
| + Log.i(TAG, "Using network stack: " + factory.getName()); |
| + return factory; |
| + } |
| + |
| + /** |
| + * Returns true if the factory is enabled. |
|
mmenke
2014/10/02 15:24:09
What does it mean for a factory to be enabled?
mmenke
2014/10/02 15:24:09
"@return True if..."
mef
2014/10/02 22:07:43
Done.
|
| + */ |
| + public abstract boolean isEnabled(); |
| + |
| + /** |
| + * Returns a human-readable name of the factory. |
|
mmenke
2014/10/02 15:24:09
@return
mef
2014/10/02 22:07:43
Done.
|
| + */ |
| + public abstract String getName(); |
| + |
| + private static UrlRequestFactory createCronetFactory(Context context, |
| + HttpUrlRequestFactoryConfig config) { |
| + UrlRequestFactory factory = null; |
| + try { |
| + Class<? extends UrlRequestFactory> factoryClass = |
| + UrlRequestFactory.class.getClassLoader(). |
| + loadClass(CRONET_URL_REQUEST_CONTEXT). |
| + asSubclass(UrlRequestFactory.class); |
| + Constructor<? extends UrlRequestFactory> constructor = |
| + factoryClass.getConstructor( |
| + Context.class, HttpUrlRequestFactoryConfig.class); |
| + UrlRequestFactory cronetFactory = |
| + constructor.newInstance(context, config); |
| + if (cronetFactory.isEnabled()) { |
| + factory = cronetFactory; |
| + } |
| + } catch (ClassNotFoundException e) { |
| + // Leave as null |
| + } catch (Exception e) { |
| + throw new IllegalStateException( |
| + "Cannot instantiate: " + |
| + CRONET_URL_REQUEST_CONTEXT, |
| + e); |
| + } |
| + return factory; |
| + } |
| } |