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

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

Issue 1412243012: Initial implementation of CronetBidirectionalStream. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address Helen's comments. Created 5 years 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/CronetUrlRequest.java
diff --git a/components/cronet/android/java/src/org/chromium/net/CronetUrlRequest.java b/components/cronet/android/java/src/org/chromium/net/CronetUrlRequest.java
index b20f810d38db52a1c8b3a339b33f36ba29880659..af2d0ad5f9ded821add2fb7904baf5a05abc594f 100644
--- a/components/cronet/android/java/src/org/chromium/net/CronetUrlRequest.java
+++ b/components/cronet/android/java/src/org/chromium/net/CronetUrlRequest.java
@@ -190,7 +190,7 @@ final class CronetUrlRequest implements UrlRequest {
try {
mUrlRequestAdapter = nativeCreateRequestAdapter(
mRequestContext.getUrlRequestContextAdapter(), mInitialUrl, mPriority);
- mRequestContext.onRequestStarted(this);
+ mRequestContext.onRequestStarted();
if (mInitialMethod != null) {
if (!nativeSetHttpMethod(mUrlRequestAdapter, mInitialMethod)) {
throw new IllegalArgumentException("Invalid http method " + mInitialMethod);
@@ -441,7 +441,7 @@ final class CronetUrlRequest implements UrlRequest {
}
nativeDestroy(mUrlRequestAdapter, sendOnCanceled);
mRequestContext.reportFinished(this);
- mRequestContext.onRequestDestroyed(this);
+ mRequestContext.onRequestDestroyed();
mUrlRequestAdapter = 0;
if (mOnDestroyedCallbackForTesting != null) {
mOnDestroyedCallbackForTesting.run();

Powered by Google App Engine
This is Rietveld 408576698