[Cronet] Add isDone() check in CronetBidirectionalStream#onWritevCompleted

components/cronet/android/java/src/org/chromium/net/impl/CronetBidirectionalStream.java

Index: components/cronet/android/java/src/org/chromium/net/impl/CronetBidirectionalStream.java
diff --git a/components/cronet/android/java/src/org/chromium/net/impl/CronetBidirectionalStream.java b/components/cronet/android/java/src/org/chromium/net/impl/CronetBidirectionalStream.java
index de0a2e71826931fef9a7ddbbb1eff6cb1a6fbbbd..624c7d15ea40622e9e57c7981868b61f5fa56e95 100644
--- a/components/cronet/android/java/src/org/chromium/net/impl/CronetBidirectionalStream.java
+++ b/components/cronet/android/java/src/org/chromium/net/impl/CronetBidirectionalStream.java
@@ -546,6 +546,7 @@ public class CronetBidirectionalStream extends ExperimentalBidirectionalStream {
         assert byteBuffers.length == initialPositions.length;
         assert byteBuffers.length == initialLimits.length;
         synchronized (mNativeStreamLock) {
+            if (isDoneLocked()) return;

[mef, 2017/02/14 21:02:47]

Just checking: Does this syntax conform to coding guidelines?

[xunjieli, 2017/02/14 21:14:34]

Yes, this is listed under "Use Standard Brace Style" on
http://source.android.com/source/code-style.html

[mef, 2017/02/14 21:19:54]

Cool, thanks for the reference!

             mWriteState = State.WAITING_FOR_FLUSH;
             // Flush if there is anything in the flush queue mFlushData.
             if (!mFlushData.isEmpty()) {