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

Side by Side Diff: net/spdy/buffered_spdy_framer.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/core/spdy_utils.cc ('k') | net/spdy/buffered_spdy_framer_unittest.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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/spdy/buffered_spdy_framer.h" 5 #include "net/spdy/buffered_spdy_framer.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/logging.h" 10 #include "base/logging.h"
11 #include "base/strings/string_util.h" 11 #include "base/strings/string_util.h"
12 12
13 namespace net { 13 namespace net {
14 14
15 namespace { 15 namespace {
16 16
17 // GOAWAY frame debug data is only buffered up to this many bytes. 17 // GOAWAY frame debug data is only buffered up to this many bytes.
18 size_t kGoAwayDebugDataMaxSize = 1024; 18 size_t kGoAwayDebugDataMaxSize = 1024;
19 19
20 } // namespace 20 } // namespace
21 21
22 BufferedSpdyFramer::BufferedSpdyFramer() 22 BufferedSpdyFramer::BufferedSpdyFramer()
23 : visitor_(NULL), 23 : spdy_framer_(SpdyFramer::ENABLE_COMPRESSION),
24 visitor_(NULL),
24 header_buffer_valid_(false), 25 header_buffer_valid_(false),
25 header_stream_id_(SpdyFramer::kInvalidStream), 26 header_stream_id_(SpdyFramer::kInvalidStream),
26 frames_received_(0) {} 27 frames_received_(0) {}
27 28
28 BufferedSpdyFramer::~BufferedSpdyFramer() { 29 BufferedSpdyFramer::~BufferedSpdyFramer() {
29 } 30 }
30 31
31 void BufferedSpdyFramer::set_visitor( 32 void BufferedSpdyFramer::set_visitor(
32 BufferedSpdyFramerVisitorInterface* visitor) { 33 BufferedSpdyFramerVisitorInterface* visitor) {
33 visitor_ = visitor; 34 visitor_ = visitor;
(...skipping 296 matching lines...) Expand 10 before | Expand all | Expand 10 after
330 } 331 }
331 332
332 void BufferedSpdyFramer::InitHeaderStreaming(SpdyStreamId stream_id) { 333 void BufferedSpdyFramer::InitHeaderStreaming(SpdyStreamId stream_id) {
333 header_buffer_.clear(); 334 header_buffer_.clear();
334 header_buffer_valid_ = true; 335 header_buffer_valid_ = true;
335 header_stream_id_ = stream_id; 336 header_stream_id_ = stream_id;
336 DCHECK_NE(header_stream_id_, SpdyFramer::kInvalidStream); 337 DCHECK_NE(header_stream_id_, SpdyFramer::kInvalidStream);
337 } 338 }
338 339
339 } // namespace net 340 } // namespace net
OLDNEW
« no previous file with comments | « net/quic/core/spdy_utils.cc ('k') | net/spdy/buffered_spdy_framer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698