Remove confusing logic in QuicChromiumClientStream::Read

net/quic/chromium/quic_chromium_client_stream.cc

Index: net/quic/chromium/quic_chromium_client_stream.cc
diff --git a/net/quic/chromium/quic_chromium_client_stream.cc b/net/quic/chromium/quic_chromium_client_stream.cc
index d224e1d761f1dabe7e591b24f8540db08a356d7b..2a3e0c04c2b312ae5d6b6f2ab2fc66a8bf018901 100644
--- a/net/quic/chromium/quic_chromium_client_stream.cc
+++ b/net/quic/chromium/quic_chromium_client_stream.cc
@@ -216,10 +216,8 @@ int QuicChromiumClientStream::Read(IOBuffer* buf, int buf_len) {
   iov.iov_base = buf->data();
   iov.iov_len = buf_len;
   size_t bytes_read = Readv(&iov, 1);
-  // If no more body bytes and trailers are to be delivered, return
-  // ERR_IO_PENDING now because onDataAvailable() will be called after trailers.
-  if (bytes_read == 0 && !FinishedReadingTrailers())
-    return ERR_IO_PENDING;
+  // Since HasBytesToRead is true, Readv() must of read some data.
+  DCHECK_NE(0u, bytes_read);
   return bytes_read;
 }