Refactor in preparation for SPDY 4, part 1.

net/spdy/spdy_frame_builder.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NET_SPDY_SPDY_FRAME_BUILDER_H_
 #define NET_SPDY_SPDY_FRAME_BUILDER_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 13 matching lines @@
 // dynamically to hold the sequence of primitive values.   The internal memory
 // buffer is exposed as the "data" of the SpdyFrameBuilder.
 class NET_EXPORT_PRIVATE SpdyFrameBuilder {
  public:
   // Initializes 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,
+  SpdyFrameBuilder(SpdyControlType type, uint8 flags,

[Ryan Hamilton, 2013/02/12 16:34:52]

After you get this all moved around and finish landing all of these internal
changes, can you add a typedef for this uint8?

[akalin, 2013/02/12 20:01:18]

Okay.  Added a TODO.

                    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);
 
   ~SpdyFrameBuilder();
 
   // Returns the size of the SpdyFrameBuilder's data.
@@ skipping 56 matching lines @@
   char* BeginWrite(size_t length);
 
   scoped_ptr<char[]> buffer_;
   size_t capacity_;  // Allocation size of payload (or -1 if buffer is const).
   size_t length_;    // current length of the buffer
 };
 
 }  // namespace net
 
 #endif  // NET_SPDY_SPDY_FRAME_BUILDER_H_