Include HTTP header bytes in GetTotalSent/ReceivedBytes for QUIC.

net/quic/quic_http_stream.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NET_QUIC_QUIC_HTTP_STREAM_H_
 #define NET_QUIC_QUIC_HTTP_STREAM_H_
 
 #include <stdint.h>
 
 #include <list>
@@ skipping 44 matching lines @@
   bool CanReuseConnection() const override;
   int64 GetTotalReceivedBytes() const override;
   int64_t GetTotalSentBytes() const override;
   bool GetLoadTimingInfo(LoadTimingInfo* load_timing_info) const override;
   void GetSSLInfo(SSLInfo* ssl_info) override;
   void GetSSLCertRequestInfo(SSLCertRequestInfo* cert_request_info) override;
   void Drain(HttpNetworkSession* session) override;
   void SetPriority(RequestPriority priority) override;
 
   // QuicReliableClientStream::Delegate implementation
-  void OnHeadersAvailable(const SpdyHeaderBlock& headers) override;
+  void OnHeadersAvailable(const SpdyHeaderBlock& headers,
+                          size_t frame_len) override;
   void OnDataAvailable() override;
   void OnClose(QuicErrorCode error) override;
   void OnError(int error) override;
   bool HasSendHeadersComplete() override;
 
   // QuicChromiumClientSession::Observer implementation
   void OnCryptoHandshakeConfirmed() override;
   void OnSessionClosed(int error) override;
 
  private:
@@ skipping 61 matching lines @@
   int response_status_;
 
   // Serialized request headers.
   SpdyHeaderBlock request_headers_;
 
   bool response_headers_received_;
 
   // Serialized HTTP request.
   std::string request_;
 
+  // Number of bytes received by the headers stream on behalf of this stream.
+  int64_t headers_stream_received_bytes_;
+  // Number of bytes sent by the headers stream on behalf of this stream.
+  int64_t headers_stream_sent_bytes_;

[Ryan Hamilton, 2015/09/23 02:53:26]

nit: I would name these: headers_bytes_sent_ and headers_bytes_received_. The
"headers stream" is a well defined thing, of course, but it will have
sent/received more bytes than those for just this stream

+
   // Number of bytes received when the stream was closed.
   int64 closed_stream_received_bytes_;
   // Number of bytes sent when the stream was closed.
   int64_t closed_stream_sent_bytes_;
 
   // The caller's callback to be used for asynchronous operations.
   CompletionCallback callback_;
 
   // Caller provided buffer for the ReadResponseBody() response.
   scoped_refptr<IOBuffer> user_buffer_;
   int user_buffer_len_;
 
   // Temporary buffer used to read the request body from UploadDataStream.
   scoped_refptr<IOBufferWithSize> raw_request_body_buf_;
   // Wraps raw_request_body_buf_ to read the remaining data progressively.
   scoped_refptr<DrainableIOBuffer> request_body_buf_;
 
   BoundNetLog stream_net_log_;
 
   base::WeakPtrFactory<QuicHttpStream> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(QuicHttpStream);
 };
 
 }  // namespace net
 
 #endif  // NET_QUIC_QUIC_HTTP_STREAM_H_