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

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

Issue 2339223002: Cronet API Refactoring (Closed)
Patch Set: Moved CronetSampleApp to AppCompat Created 4 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/api/src/org/chromium/net/HttpUrlConnectionUrlRequestFactory.java
diff --git a/components/cronet/android/api/src/org/chromium/net/HttpUrlConnectionUrlRequestFactory.java b/components/cronet/android/api/src/org/chromium/net/HttpUrlConnectionUrlRequestFactory.java
deleted file mode 100644
index 238f2915e49b6d193c865f404482a79ebdea1553..0000000000000000000000000000000000000000
--- a/components/cronet/android/api/src/org/chromium/net/HttpUrlConnectionUrlRequestFactory.java
+++ /dev/null
@@ -1,74 +0,0 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-package org.chromium.net;
-
-import android.content.Context;
-
-import java.io.IOException;
-import java.io.PrintWriter;
-import java.nio.channels.WritableByteChannel;
-import java.util.Map;
-
-/**
- * Network request using {@link java.net.HttpURLConnection}.
- * @deprecated Use {@link CronetEngine} instead.
- * {@hide as it's deprecated}
- */
-@Deprecated
-class HttpUrlConnectionUrlRequestFactory extends HttpUrlRequestFactory {
-
- private final Context mContext;
- private final String mDefaultUserAgent;
-
- public HttpUrlConnectionUrlRequestFactory(Context context, CronetEngine.Builder config) {
- mContext = context;
- String userAgent = config.getUserAgent();
- if (userAgent == null) {
- // Cannot use config.getDefaultUserAgent() as config.mContext may be null.
- userAgent = new CronetEngine.Builder(mContext).getDefaultUserAgent();
- }
- mDefaultUserAgent = userAgent;
- }
-
- @Override
- public boolean isEnabled() {
- return true;
- }
-
- @Override
- public String getName() {
- return "HttpUrlConnection/" + ApiVersion.getVersion();
- }
-
- @Override
- public HttpUrlRequest createRequest(String url, int requestPriority,
- Map<String, String> headers, HttpUrlRequestListener listener) {
- return new HttpUrlConnectionUrlRequest(mContext, mDefaultUserAgent, url,
- requestPriority, headers, listener);
- }
-
- @Override
- public HttpUrlRequest createRequest(String url, int requestPriority,
- Map<String, String> headers, WritableByteChannel channel,
- HttpUrlRequestListener listener) {
- return new HttpUrlConnectionUrlRequest(mContext, mDefaultUserAgent, url,
- requestPriority, headers, channel, listener);
- }
-
- @Override
- public void startNetLogToFile(String fileName, boolean logAll) {
- try {
- PrintWriter out = new PrintWriter(fileName);
- out.println("NetLog is not supported by " + getName());
- out.close();
- } catch (IOException e) {
- // Ignore any exceptions.
- }
- }
-
- @Override
- public void stopNetLog() {
- }
-}

Powered by Google App Engine
This is Rietveld 408576698