Fix QuicHttpStream reporting negative sent bytes.

net/quic/chromium/quic_http_stream_test.cc

Index: net/quic/chromium/quic_http_stream_test.cc
diff --git a/net/quic/chromium/quic_http_stream_test.cc b/net/quic/chromium/quic_http_stream_test.cc
index d22ea64ecf1e0d64f925bda47f04a0e721c8d66b..9e48dc9ba7a6a62566456ae36313d98d2bd72232 100644
--- a/net/quic/chromium/quic_http_stream_test.cc
+++ b/net/quic/chromium/quic_http_stream_test.cc
@@ -1530,6 +1530,9 @@ TEST_P(QuicHttpStreamTest, SessionClosedBeforeSendHeadersComplete) {
                                       net_log_.bound(), callback_.callback()));
   ASSERT_EQ(ERR_QUIC_PROTOCOL_ERROR,
             stream_->SendRequest(headers_, &response_, callback_.callback()));
+
+  EXPECT_LE(0, stream_->GetTotalSentBytes());

[mmenke, 2017/06/14 21:20:41]

Speaking of which...We don't have any async version of these tests.  Should we?

[mmenke, 2017/06/14 21:20:41]

This could be EQ, since the writes are synchronous, but I'm paranoid.

[Zhongyi Shi, 2017/06/15 17:32:19]

I think you're right. It might be covered implicitly in some other tests
already. But, yeah, no async version of these unit tests :( 

+  EXPECT_EQ(0, stream_->GetTotalReceivedBytes());
 }
 
 TEST_P(QuicHttpStreamTest, SessionClosedBeforeSendBodyComplete) {