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

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

Issue 2339223002: Cronet API Refactoring (Closed)
Patch Set: Javadoc + rebase 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/java/src/org/chromium/net/impl/CronetUrlRequest.java
diff --git a/components/cronet/android/java/src/org/chromium/net/impl/CronetUrlRequest.java b/components/cronet/android/java/src/org/chromium/net/impl/CronetUrlRequest.java
index 3dea66d9085208e29436dffd68f3b8e9091f3479..3e88620fc5a54017a5db2e14f3ec54b9a90abe5e 100644
--- a/components/cronet/android/java/src/org/chromium/net/impl/CronetUrlRequest.java
+++ b/components/cronet/android/java/src/org/chromium/net/impl/CronetUrlRequest.java
@@ -14,7 +14,6 @@ import org.chromium.base.annotations.JNIAdditionalImport;
import org.chromium.base.annotations.JNINamespace;
import org.chromium.base.annotations.NativeClassQualifiedName;
import org.chromium.net.InlineExecutionProhibitedException;
-import org.chromium.net.Preconditions;
import org.chromium.net.QuicException;
import org.chromium.net.RequestFinishedInfo;
import org.chromium.net.RequestPriority;
@@ -47,7 +46,7 @@ import javax.annotation.concurrent.GuardedBy;
// Qualifies UrlRequest.StatusListener which is used in onStatus, a JNI method.
@JNIAdditionalImport(UrlRequest.class)
@VisibleForTesting
-public final class CronetUrlRequest implements UrlRequest {
+public final class CronetUrlRequest extends UrlRequestBase {
private static final RequestFinishedInfo.Metrics EMPTY_METRICS =
new RequestFinishedInfo.Metrics(null, null, null, null);
private final boolean mAllowDirectExecutor;
@@ -693,7 +692,7 @@ public final class CronetUrlRequest implements UrlRequest {
Runnable task = new Runnable() {
@Override
public void run() {
- listener.onStatus(UrlRequest.Status.convertLoadState(loadState));
+ listener.onStatus(UrlRequestBase.Status.convertLoadState(loadState));
}
};
postTaskToExecutor(task);

Powered by Google App Engine
This is Rietveld 408576698