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

Side by Side Diff: net/spdy/buffered_spdy_framer.cc

Issue 11415219: Move a number of static variables SPDY to HttpNetworkSession::Params. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Moar cleanup Created 8 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 | Annotate | Revision Log
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 "base/logging.h" 7 #include "base/logging.h"
8 8
9 namespace {
10
11 bool g_enable_compression_default = true;
12
13 } // namespace
14
15 namespace net { 9 namespace net {
16 10
17 BufferedSpdyFramer::BufferedSpdyFramer(int version) 11 BufferedSpdyFramer::BufferedSpdyFramer(int version, bool enable_compression)
18 : spdy_framer_(version), 12 : spdy_framer_(version),
19 visitor_(NULL), 13 visitor_(NULL),
20 header_buffer_used_(0), 14 header_buffer_used_(0),
21 header_buffer_valid_(false), 15 header_buffer_valid_(false),
22 header_stream_id_(SpdyFramer::kInvalidStream), 16 header_stream_id_(SpdyFramer::kInvalidStream),
23 frames_received_(0) { 17 frames_received_(0) {
24 spdy_framer_.set_enable_compression(g_enable_compression_default); 18 //DCHECK(!enable_compression);
willchan no longer on Chromium 2012/12/01 20:49:13 ?
Ryan Hamilton 2012/12/01 23:02:25 Removed. (Sorry, left over from debugging)
19 spdy_framer_.set_enable_compression(enable_compression);
25 memset(header_buffer_, 0, sizeof(header_buffer_)); 20 memset(header_buffer_, 0, sizeof(header_buffer_));
26 } 21 }
27 22
28 BufferedSpdyFramer::~BufferedSpdyFramer() { 23 BufferedSpdyFramer::~BufferedSpdyFramer() {
29 } 24 }
30 25
31 void BufferedSpdyFramer::set_visitor( 26 void BufferedSpdyFramer::set_visitor(
32 BufferedSpdyFramerVisitorInterface* visitor) { 27 BufferedSpdyFramerVisitorInterface* visitor) {
33 visitor_ = visitor; 28 visitor_ = visitor;
34 spdy_framer_.set_visitor(this); 29 spdy_framer_.set_visitor(this);
(...skipping 254 matching lines...) Expand 10 before | Expand all | Expand 10 after
289 } 284 }
290 285
291 SpdyPriority BufferedSpdyFramer::GetHighestPriority() const { 286 SpdyPriority BufferedSpdyFramer::GetHighestPriority() const {
292 return spdy_framer_.GetHighestPriority(); 287 return spdy_framer_.GetHighestPriority();
293 } 288 }
294 289
295 bool BufferedSpdyFramer::IsCompressible(const SpdyFrame& frame) const { 290 bool BufferedSpdyFramer::IsCompressible(const SpdyFrame& frame) const {
296 return spdy_framer_.IsCompressible(frame); 291 return spdy_framer_.IsCompressible(frame);
297 } 292 }
298 293
299 // static
300 void BufferedSpdyFramer::set_enable_compression_default(bool value) {
301 g_enable_compression_default = value;
302 }
303
304 void BufferedSpdyFramer::InitHeaderStreaming(SpdyStreamId stream_id) { 294 void BufferedSpdyFramer::InitHeaderStreaming(SpdyStreamId stream_id) {
305 memset(header_buffer_, 0, kHeaderBufferSize); 295 memset(header_buffer_, 0, kHeaderBufferSize);
306 header_buffer_used_ = 0; 296 header_buffer_used_ = 0;
307 header_buffer_valid_ = true; 297 header_buffer_valid_ = true;
308 header_stream_id_ = stream_id; 298 header_stream_id_ = stream_id;
309 DCHECK_NE(header_stream_id_, SpdyFramer::kInvalidStream); 299 DCHECK_NE(header_stream_id_, SpdyFramer::kInvalidStream);
310 } 300 }
311 301
312 } // namespace net 302 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698