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

Unified Diff: net/spdy/spdy_frame_builder.h

Issue 10053029: Factor out the code to write control frame headers into a new constructor of SpdyFrameBuilder. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add Data frame constructor 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
« no previous file with comments | « no previous file | net/spdy/spdy_frame_builder.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_frame_builder.h
diff --git a/net/spdy/spdy_frame_builder.h b/net/spdy/spdy_frame_builder.h
index 31d0211d7ae4a510eba99582d7c9351df5c2dc58..ed7c78c7bbb40e861582bd07cd0228dc887187d1 100644
--- a/net/spdy/spdy_frame_builder.h
+++ b/net/spdy/spdy_frame_builder.h
@@ -27,10 +27,16 @@ class NET_EXPORT_PRIVATE SpdyFrameBuilder {
public:
~SpdyFrameBuilder();
- SpdyFrameBuilder();
-
- // Initiailizes a SpdyFrameBuilder with a buffer of given size.
- explicit SpdyFrameBuilder(size_t size);
+ // Initializes a SpdyFrameBuilder with a buffer of given size,
+ // populate with a SPDY control frame header based
+ // on |type|, |flags|, and |spdy_version|.
+ SpdyFrameBuilder(SpdyControlType type, SpdyControlFlags flags,
+ int spdy_version, size_t size);
+
+ // Initiailizes a SpdyFrameBuilder with a buffer of given size,
+ // populated with a SPDY data frame header based on
+ // |stream_id| and |flags|.
+ SpdyFrameBuilder(SpdyStreamId stream_id, SpdyDataFlags flags, size_t size);
// Returns the size of the SpdyFrameBuilder's data.
int length() const { return length_; }
« no previous file with comments | « no previous file | net/spdy/spdy_frame_builder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698