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

Unified Diff: components/cronet/android/java/src/org/chromium/net/urlconnection/CronetBufferedOutputStream.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/urlconnection/CronetBufferedOutputStream.java
diff --git a/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetBufferedOutputStream.java b/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetBufferedOutputStream.java
index a7db52784d8ea4c0c564c3b691dc19035aac97b5..f92dfc5cb048a30e580c8f4b87c53dfe4da847a8 100644
--- a/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetBufferedOutputStream.java
+++ b/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetBufferedOutputStream.java
@@ -146,7 +146,7 @@ final class CronetBufferedOutputStream extends CronetOutputStream {
// TODO(xunjieli): Think of a less fragile way, since getLength() can be
// potentially called in other places in the future.
if (mInitialContentLength == -1) {
- return mBuffer.position();
+ return mConnected ? mBuffer.limit() : mBuffer.position();
mef 2015/09/25 21:32:19 Um, can you elaborate on this? Seems a bit magical
pauljensen 2015/09/28 14:18:12 Done.
}
return mInitialContentLength;
}

Powered by Google App Engine
This is Rietveld 408576698