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

Unified Diff: net/spdy/core/spdy_protocol.cc

Issue 2849713002: Remove one layer of abstraction from HTTP/2 frame representations. (Closed)
Patch Set: Created 3 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
« no previous file with comments | « net/spdy/core/spdy_protocol.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/core/spdy_protocol.cc
diff --git a/net/spdy/core/spdy_protocol.cc b/net/spdy/core/spdy_protocol.cc
index 2ae97a9bf906d2c23052f54451c78bc371b0be2a..9a0f52beb05af03ade9efa33c860a7aae6504c88 100644
--- a/net/spdy/core/spdy_protocol.cc
+++ b/net/spdy/core/spdy_protocol.cc
@@ -255,7 +255,7 @@ SpdyFrameType SpdyDataIR::frame_type() const {
SpdyRstStreamIR::SpdyRstStreamIR(SpdyStreamId stream_id,
SpdyErrorCode error_code)
- : SpdyFrameWithStreamIdIR(stream_id) {
+ : SpdyFrameIR(stream_id) {
set_error_code(error_code);
}
@@ -324,7 +324,7 @@ SpdyFrameType SpdyGoAwayIR::frame_type() const {
}
SpdyContinuationIR::SpdyContinuationIR(SpdyStreamId stream_id)
- : SpdyFrameWithStreamIdIR(stream_id), end_headers_(false) {
+ : SpdyFrameIR(stream_id), end_headers_(false) {
encoding_ = SpdyMakeUnique<SpdyString>();
}
@@ -362,9 +362,7 @@ SpdyFrameType SpdyPushPromiseIR::frame_type() const {
return SpdyFrameType::PUSH_PROMISE;
}
-SpdyAltSvcIR::SpdyAltSvcIR(SpdyStreamId stream_id)
- : SpdyFrameWithStreamIdIR(stream_id) {
-}
+SpdyAltSvcIR::SpdyAltSvcIR(SpdyStreamId stream_id) : SpdyFrameIR(stream_id) {}
SpdyAltSvcIR::~SpdyAltSvcIR() {
}
« no previous file with comments | « net/spdy/core/spdy_protocol.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698