Replace WebSocketFrameChunk with WebSocketFrame

net/websockets/websocket_frame.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_WEBSOCKETS_WEBSOCKET_FRAME_H_
 #define NET_WEBSOCKETS_WEBSOCKET_FRAME_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "net/base/net_export.h"
 
 namespace net {
 
+class IOBuffer;
 class IOBufferWithSize;
 
 // Represents a WebSocket frame header.
 //
 // Members of this class correspond to each element in WebSocket frame header
 // (see http://tools.ietf.org/html/rfc6455#section-5.2).
 struct NET_EXPORT WebSocketFrameHeader {
   typedef int OpCode;
 
   // Originally these constants were static const int, but to make it possible
@@ skipping 35 matching lines @@
   explicit WebSocketFrameHeader(OpCode opCode)
       : final(false),
         reserved1(false),
         reserved2(false),
         reserved3(false),
         opcode(opCode),
         masked(false),
         payload_length(0) {}
 
   // Create a clone of this object on the heap.
-  scoped_ptr<WebSocketFrameHeader> Clone();
+  scoped_ptr<WebSocketFrameHeader> Clone() const;
+
+  // Overwrite this object with the fields from |source|.
+  void CopyFrom(const WebSocketFrameHeader& source);
 
   // Members below correspond to each item in WebSocket frame header.
   // See <http://tools.ietf.org/html/rfc6455#section-5.2> for details.
   bool final;
   bool reserved1;
   bool reserved2;
   bool reserved3;
   OpCode opcode;
   bool masked;
   uint64 payload_length;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(WebSocketFrameHeader);
 };
 
-// Contains payload data of part of a WebSocket frame.
+// Contains an entire WebSocket frame including payload. This is used by APIs
+// that are not concerned about retaining the original frame boundaries (because
+// frames may need to be split in order for the data to fit in memory).
+struct NET_EXPORT_PRIVATE WebSocketFrame {
+  // A frame must always have an opcode, so this parameter is compulsory.
+  explicit WebSocketFrame(WebSocketFrameHeader::OpCode opcode);
+  ~WebSocketFrame();
+
+  // |header| is always present.
+  WebSocketFrameHeader header;
+
+  // |data| is always unmasked even if the frame is masked. The size of |data|
+  // |is given by |header.payload_length|.

[tyoshino (SeeGerritForStatus), 2013/09/25 08:39:23]

|is -> is

[tyoshino (SeeGerritForStatus), 2013/09/26 06:00:25]

And, please fix this.

[Adam Rice, 2013/09/26 06:25:55]

Sorry! Fixed it now.

+  scoped_refptr<IOBuffer> data;
+};
+
+// Structure describing one chunk of a WebSocket frame.
 //
-// Payload of a WebSocket frame may be divided into multiple chunks.
+// The payload of a WebSocket frame may be divided into multiple chunks.
 // You need to look at |final_chunk| member variable to detect the end of a
 // series of chunk objects of a WebSocket frame.
 //
-// Frame dissection is necessary to handle WebSocket frame stream containing
-// abritrarily large frames in the browser process. Because the server may send
-// a huge frame that doesn't fit in the memory, we cannot store the entire
-// payload data in the memory.
+// Frame dissection is necessary to handle frames that are too large to store in
+// the browser memory without losing information about the frame boundaries. In
+// practice, most code does not need to worry about the original frame
+// boundaries and can use the WebSocketFrame type declared above.
 //
 // Users of this struct should treat WebSocket frames as a data stream; it's
 // important to keep the frame data flowing, especially in the browser process.
 // Users should not let the data stuck somewhere in the pipeline.
 //
 // This struct is used for reading WebSocket frame data (created by
 // WebSocketFrameParser). To construct WebSocket frames, use functions below.
 struct NET_EXPORT WebSocketFrameChunk {
   WebSocketFrameChunk();
   ~WebSocketFrameChunk();
@@ skipping 58 matching lines @@
 // used for unmasking a WebSocket frame.
 NET_EXPORT void MaskWebSocketFramePayload(
     const WebSocketMaskingKey& masking_key,
     uint64 frame_offset,
     char* data,
     int data_size);
 
 }  // namespace net
 
 #endif  // NET_WEBSOCKETS_WEBSOCKET_FRAME_H_