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

Side by Side Diff: net/quic/core/quic_headers_stream.cc

Issue 2566513004: This change removes SpdyFramer::set_enable_compression() in favor of a constructor parameter. (Closed)
Patch Set: Created 4 years 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 unified diff | Download patch
« no previous file with comments | « net/quic/chromium/quic_test_packet_maker.cc ('k') | net/quic/core/quic_headers_stream_test.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/quic/core/quic_headers_stream.h" 5 #include "net/quic/core/quic_headers_stream.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 10
(...skipping 309 matching lines...) Expand 10 before | Expand all | Expand 10 after
320 : QuicStream(kHeadersStreamId, session), 320 : QuicStream(kHeadersStreamId, session),
321 spdy_session_(session), 321 spdy_session_(session),
322 stream_id_(kInvalidStreamId), 322 stream_id_(kInvalidStreamId),
323 promised_stream_id_(kInvalidStreamId), 323 promised_stream_id_(kInvalidStreamId),
324 fin_(false), 324 fin_(false),
325 frame_len_(0), 325 frame_len_(0),
326 uncompressed_frame_len_(0), 326 uncompressed_frame_len_(0),
327 supports_push_promise_(session->perspective() == Perspective::IS_CLIENT), 327 supports_push_promise_(session->perspective() == Perspective::IS_CLIENT),
328 cur_max_timestamp_(QuicTime::Zero()), 328 cur_max_timestamp_(QuicTime::Zero()),
329 prev_max_timestamp_(QuicTime::Zero()), 329 prev_max_timestamp_(QuicTime::Zero()),
330 spdy_framer_(SpdyFramer::ENABLE_COMPRESSION),
330 spdy_framer_visitor_(new SpdyFramerVisitor(this)) { 331 spdy_framer_visitor_(new SpdyFramerVisitor(this)) {
331 spdy_framer_.set_visitor(spdy_framer_visitor_.get()); 332 spdy_framer_.set_visitor(spdy_framer_visitor_.get());
332 spdy_framer_.set_debug_visitor(spdy_framer_visitor_.get()); 333 spdy_framer_.set_debug_visitor(spdy_framer_visitor_.get());
333 // The headers stream is exempt from connection level flow control. 334 // The headers stream is exempt from connection level flow control.
334 DisableConnectionFlowControlForThisStream(); 335 DisableConnectionFlowControlForThisStream();
335 } 336 }
336 337
337 QuicHeadersStream::~QuicHeadersStream() {} 338 QuicHeadersStream::~QuicHeadersStream() {}
338 339
339 size_t QuicHeadersStream::WriteHeaders(QuicStreamId stream_id, 340 size_t QuicHeadersStream::WriteHeaders(QuicStreamId stream_id,
(...skipping 282 matching lines...) Expand 10 before | Expand all | Expand 10 after
622 return true; 623 return true;
623 } 624 }
624 frame_len_ -= len; 625 frame_len_ -= len;
625 // Ignore fin_ while there is more data coming, if frame_len_ > 0. 626 // Ignore fin_ while there is more data coming, if frame_len_ > 0.
626 spdy_session_->OnStreamFrameData(stream_id, data, len, 627 spdy_session_->OnStreamFrameData(stream_id, data, len,
627 frame_len_ > 0 ? false : fin_); 628 frame_len_ > 0 ? false : fin_);
628 return true; 629 return true;
629 } 630 }
630 631
631 } // namespace net 632 } // namespace net
OLDNEW
« no previous file with comments | « net/quic/chromium/quic_test_packet_maker.cc ('k') | net/quic/core/quic_headers_stream_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698