Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(359)

Unified Diff: net/quic/chromium/quic_http_stream.h

Issue 2900533002: Add an async ReadTrailers method to QuicChromiumClientStream::Handle (Closed)
Patch Set: Rebase Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/quic/chromium/quic_http_stream.h
diff --git a/net/quic/chromium/quic_http_stream.h b/net/quic/chromium/quic_http_stream.h
index be8833c4a5796aac8e578153389e930bb7020e70..562a076d27a3d3dcd85d310b0dece04d50dde430 100644
--- a/net/quic/chromium/quic_http_stream.h
+++ b/net/quic/chromium/quic_http_stream.h
@@ -69,8 +69,6 @@ class NET_EXPORT_PRIVATE QuicHttpStream
void SetPriority(RequestPriority priority) override;
// QuicChromiumClientStream::Delegate implementation
- void OnTrailingHeadersAvailable(const SpdyHeaderBlock& headers,
- size_t frame_len) override;
void OnClose() override;
void OnError(int error) override;
@@ -123,6 +121,8 @@ class NET_EXPORT_PRIVATE QuicHttpStream
void OnReadResponseHeadersComplete(int rv);
int ProcessResponseHeaders(const SpdyHeaderBlock& headers);
+ void ReadTrailingHeaders();
+ void OnReadTrailingHeadersComplete(int rv);
void OnReadBodyComplete(int rv);
int HandleReadComplete(int rv);
@@ -184,6 +184,9 @@ class NET_EXPORT_PRIVATE QuicHttpStream
SpdyHeaderBlock response_header_block_;
bool response_headers_received_;
+ SpdyHeaderBlock trailing_header_block_;
+ bool trailing_headers_received_;
xunjieli 2017/05/26 15:53:06 nit: we need to default initialize this boolean.
Ryan Hamilton 2017/05/26 22:35:50 Done.
+
// Number of bytes received by the headers stream on behalf of this stream.
int64_t headers_bytes_received_;
// Number of bytes sent by the headers stream on behalf of this stream.

Powered by Google App Engine
This is Rietveld 408576698