Simplify CombineFrames() interface.

net/spdy/chromium/spdy_test_util_common.h

 // Copyright (c) 2013 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_CHROMIUM_SPDY_TEST_UTIL_COMMON_H_
 #define NET_SPDY_CHROMIUM_SPDY_TEST_UTIL_COMMON_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 73 matching lines @@
 
 // Create a MockRead from the given SpdySerializedFrame.
 MockRead CreateMockRead(const SpdySerializedFrame& resp);
 
 // Create a MockRead from the given SpdySerializedFrame and sequence number.
 MockRead CreateMockRead(const SpdySerializedFrame& resp, int seq);
 
 MockRead CreateMockRead(const SpdySerializedFrame& resp, int seq, IoMode mode);
 
 // Combines the given SpdySerializedFrame into the given char array and returns
 // the total length.

[Zhongyi Shi, 2017/06/12 19:34:12]

Could you update the comments since the return type is no longer the total
length?

[Bence, 2017/06/12 19:49:13]

Oops, sorry about that.  Good catch!

-int CombineFrames(const SpdySerializedFrame** frames,
-                  int num_frames,
-                  char* buf,
-                  int buf_len);
+SpdySerializedFrame CombineFrames(
+    std::vector<const SpdySerializedFrame*> frames);
 
 // Returns the SpdyPriority embedded in the given frame.  Returns true
 // and fills in |priority| on success.
 bool GetSpdyPriority(const SpdySerializedFrame& frame, SpdyPriority* priority);
 
 // Tries to create a stream in |session| synchronously. Returns NULL
 // on failure.
 base::WeakPtr<SpdyStream> CreateStreamSynchronously(
     SpdyStreamType type,
     const base::WeakPtr<SpdySession>& session,
@@ skipping 424 matching lines @@
 
   GURL default_url_;
 
   // Track a FIFO list of the stream_id of all created requests by priority.
   std::map<int, std::vector<int>> priority_to_stream_id_list_;
 };
 
 }  // namespace net
 
 #endif  // NET_SPDY_CHROMIUM_SPDY_TEST_UTIL_COMMON_H_