Chromium Code Reviews| Index: components/cronet/android/java/src/org/chromium/net/urlconnection/CronetChunkedOutputStream.java |
| diff --git a/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetChunkedOutputStream.java b/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetChunkedOutputStream.java |
| index dd0d53c041fbc2bda2e31cfa9665df2c3f9673c7..d2636cc79485876457917d31d33be914758c2603 100644 |
| --- a/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetChunkedOutputStream.java |
| +++ b/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetChunkedOutputStream.java |
| @@ -85,7 +85,6 @@ final class CronetChunkedOutputStream extends CronetOutputStream { |
| if (!mLastChunk) { |
|
pauljensen
2016/02/04 15:27:26
I think we can get rid of this if-statement now.
xunjieli
2016/07/08 14:02:14
Done.
|
| // Write last chunk. |
| mLastChunk = true; |
| - mMessageLoop.loop(); |
| } |
| mClosed = true; |
| } |
| @@ -137,9 +136,11 @@ final class CronetChunkedOutputStream extends CronetOutputStream { |
| byteBuffer.put(mBuffer); |
| // Reuse this buffer. |
| mBuffer.clear(); |
| - // Quit message loop so embedder can write more data. |
| - mMessageLoop.quit(); |
| uploadDataSink.onReadSucceeded(mLastChunk); |
| + if (!mLastChunk) { |
| + // Quit message loop so embedder can write more data. |
| + mMessageLoop.quit(); |
| + } |
| } |
| } |