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

Unified Diff: net/tools/quic/quic_simple_server_stream.cc

Issue 1910593004: Plumbs new SpdyFramerVisitorInterface method handling through QUIC test (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@119562680
Patch Set: Rebase Created 4 years, 8 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/tools/quic/quic_simple_server_stream.cc
diff --git a/net/tools/quic/quic_simple_server_stream.cc b/net/tools/quic/quic_simple_server_stream.cc
index 500754d63304e0dddd4e8b54495387ef3018a955..21a7a8fd2adddda326d9e036a2e0f9cddf271dff 100644
--- a/net/tools/quic/quic_simple_server_stream.cc
+++ b/net/tools/quic/quic_simple_server_stream.cc
@@ -43,12 +43,33 @@ void QuicSimpleServerStream::OnInitialHeadersComplete(bool fin,
MarkHeadersConsumed(decompressed_headers().length());
}
+void QuicSimpleServerStream::OnInitialHeadersComplete(
+ bool fin,
+ size_t frame_len,
+ const QuicHeaderList& header_list) {
+ QuicSpdyStream::OnInitialHeadersComplete(fin, frame_len, header_list);
+ if (!SpdyUtils::CopyAndValidateHeaders(header_list, &content_length_,
+ &request_headers_)) {
+ DVLOG(1) << "Invalid headers";
+ SendErrorResponse();
+ }
+ ConsumeHeaderList();
+}
+
void QuicSimpleServerStream::OnTrailingHeadersComplete(bool fin,
size_t frame_len) {
QUIC_BUG << "Server does not support receiving Trailers.";
SendErrorResponse();
}
+void QuicSimpleServerStream::OnTrailingHeadersComplete(
+ bool fin,
+ size_t frame_len,
+ const QuicHeaderList& header_list) {
+ QUIC_BUG << "Server does not support receiving Trailers.";
+ SendErrorResponse();
+}
+
void QuicSimpleServerStream::OnDataAvailable() {
while (HasBytesToRead()) {
struct iovec iov;

Powered by Google App Engine
This is Rietveld 408576698