QuicServer: Use Chrome's header parsing rather than Balsa

net/quic/quic_spdy_server_stream.h

Index: net/quic/quic_spdy_server_stream.h
diff --git a/net/quic/quic_spdy_server_stream.h b/net/quic/quic_spdy_server_stream.h
index 31d750cf6987aae3ddd5ba9298e7df813e5a9db9..38842662edf83abe4d4239440162f50d9168bfb8 100644
--- a/net/quic/quic_spdy_server_stream.h
+++ b/net/quic/quic_spdy_server_stream.h
@@ -9,9 +9,10 @@
 
 #include "base/basictypes.h"
 #include "net/base/io_buffer.h"
+#include "net/http/http_response_headers.h"

[wtc, 2014/06/25 22:22:24]

A forward declaration of class HttpResponseHeaders should be enough (because of
the change I suggest below).

[dmziegler, 2014/06/26 01:23:29]

Done.

 #include "net/quic/quic_data_stream.h"
 #include "net/quic/quic_protocol.h"
-#include "net/tools/balsa/balsa_headers.h"
+#include "url/gurl.h"
 
 namespace net {
 
@@ -46,10 +47,11 @@ class QuicSpdyServerStream : public QuicDataStream {
   // for the body
   void SendErrorResponse();
 
-  void SendHeadersAndBody(const BalsaHeaders& response_headers,
+  void SendHeadersAndBody(scoped_refptr<HttpResponseHeaders> response_headers,

[wtc, 2014/06/25 22:22:24]

Change this to
   const HttpResponseHeaders& response_headers,

Passing a scoped_refptr parameter implies transferring a reference to the
method.

[dmziegler, 2014/06/26 01:23:29]

Done.

                           base::StringPiece body);
 
-  BalsaHeaders headers_;
+  SpdyHeaderBlock headers_;
+  GURL request_url_;

[wtc, 2014/06/25 22:22:24]

Nit: move request_url_ forward, so that we don't separate headers_ and body_.

[dmziegler, 2014/06/26 01:23:29]

Done.

   string body_;
 
   // Buffer into which response header data is read.