Switch CreateSpdyHeadersFromHttpRequest to construct the correct headers based on the spdy protocol…

net/spdy/spdy_session_spdy2_unittest.cc

Index: net/spdy/spdy_session_spdy2_unittest.cc
diff --git a/net/spdy/spdy_session_spdy2_unittest.cc b/net/spdy/spdy_session_spdy2_unittest.cc
index 5d85f8f8be0c3479d36dabf2c59ecf49cb68e8c8..f783b8b7c0cc03dfc659a25062297bba50ab736d 100644
--- a/net/spdy/spdy_session_spdy2_unittest.cc
+++ b/net/spdy/spdy_session_spdy2_unittest.cc
@@ -127,6 +127,7 @@ TEST_F(SpdySessionSpdy2Test, GoAway) {
   session_deps.socket_factory->AddSocketDataProvider(&data);
 
   SSLSocketDataProvider ssl(SYNCHRONOUS, OK);
+  ssl.SetNextProto(SSLClientSocket::kProtoSPDY21);

[ramant (doing other things), 2012/03/15 05:41:34]

kProtoSPDY21 is this intentional? Could we use kProtoSPDY2 because it is SPDY/2
tests?

[Ryan Hamilton, 2012/03/15 16:38:06]

Done.

   session_deps.socket_factory->AddSSLSocketDataProvider(&ssl);
 
   scoped_refptr<HttpNetworkSession> http_session(
@@ -154,6 +155,7 @@ TEST_F(SpdySessionSpdy2Test, GoAway) {
                                  http_session->GetTransportSocketPool(),
                                  BoundNetLog()));
   EXPECT_EQ(OK, session->InitializeWithSocket(connection.release(), false, OK));
+  EXPECT_EQ(2, session->GetProtocolVersion());
 
   // Flush the SpdySession::OnReadComplete() task.
   MessageLoop::current()->RunAllPending();