Added a net::BidirectionalStream to expose a bidirectional streaming interface

net/spdy/spdy_test_util_common.cc

Index: net/spdy/spdy_test_util_common.cc
diff --git a/net/spdy/spdy_test_util_common.cc b/net/spdy/spdy_test_util_common.cc
index 8225d0aa7de63483a874ee042e64ee4f8776655e..4c65a4b8e8032406cb65b9969ae194a7a07f1ef9 100644
--- a/net/spdy/spdy_test_util_common.cc
+++ b/net/spdy/spdy_test_util_common.cc
@@ -1100,7 +1100,15 @@ SpdyFrame* SpdyTestUtil::ConstructSpdyPushHeaders(
 SpdyFrame* SpdyTestUtil::ConstructSpdyHeaderFrame(int stream_id,
                                                   const char* const headers[],
                                                   int header_count) {
+  return ConstructSpdyHeaderFrame(stream_id, headers, header_count, false);
+}
+
+SpdyFrame* SpdyTestUtil::ConstructSpdyHeaderFrame(int stream_id,
+                                                  const char* const headers[],
+                                                  int header_count,
+                                                  bool fin) {
   SpdyHeadersIR spdy_headers(stream_id);
+  spdy_headers.set_fin(fin);
   AppendToHeaderBlock(headers, header_count,
                       spdy_headers.mutable_header_block());
   return CreateFramer(false)->SerializeFrame(spdy_headers);
@@ -1225,7 +1233,7 @@ SpdyFrame* SpdyTestUtil::ConstructSpdyPostSynReply(
     const char* const extra_headers[],
     int extra_header_count) {
   // TODO(jgraettinger): Remove this method.
-  return ConstructSpdyGetSynReply(NULL, 0, 1);
+  return ConstructSpdyGetSynReply(extra_headers, extra_header_count, 1);

[mef, 2015/12/21 17:31:43]

why this changed?

[xunjieli, 2015/12/21 22:01:22]

I was trying to test setting extra headers on the HEADER block, as recommended
by Bence. The method wasn't passing |extra_headers| argument correctly.

[mef, 2015/12/21 23:21:31]

Acknowledged.

 }
 
 SpdyFrame* SpdyTestUtil::ConstructSpdyBodyFrame(int stream_id, bool fin) {