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

Unified Diff: net/spdy/spdy_session.cc

Issue 10138003: Remove SpdyFramer::CompressFrame() and use CompressControlFrame instead. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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
« net/spdy/spdy_framer.cc ('K') | « net/spdy/spdy_framer_test.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_session.cc
diff --git a/net/spdy/spdy_session.cc b/net/spdy/spdy_session.cc
index be67c993506402d5d4d75a53da3d3898c25f495c..278fa25ac6f25bf9c86c0bdbcd37d573dc187598 100644
--- a/net/spdy/spdy_session.cc
+++ b/net/spdy/spdy_session.cc
@@ -1035,8 +1035,10 @@ void SpdySession::WriteSocket() {
SpdyFrame uncompressed_frame(next_buffer.buffer()->data(), false);
size_t size;
if (buffered_spdy_framer_->IsCompressible(uncompressed_frame)) {
+ DCHECK(uncompressed_frame.is_control_frame());
scoped_ptr<SpdyFrame> compressed_frame(
- buffered_spdy_framer_->CompressFrame(uncompressed_frame));
+ buffered_spdy_framer_->CompressControlFrame(
+ reinterpret_cast<const SpdyControlFrame&>(uncompressed_frame)));
if (!compressed_frame.get()) {
CloseSessionOnError(
net::ERR_SPDY_PROTOCOL_ERROR, true, "SPDY Compression failure.");
« net/spdy/spdy_framer.cc ('K') | « net/spdy/spdy_framer_test.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698