Add SPDY request headers to DevTools.

content/browser/debugger/devtools_netlog_observer.cc

Index: content/browser/debugger/devtools_netlog_observer.cc
===================================================================
--- content/browser/debugger/devtools_netlog_observer.cc	(revision 154321)
+++ content/browser/debugger/devtools_netlog_observer.cc	(working copy)
@@ -13,6 +13,7 @@
 #include "net/base/load_flags.h"
 #include "net/http/http_response_headers.h"
 #include "net/http/http_util.h"
+#include "net/spdy/spdy_header_block.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_netlog_params.h"
 #include "webkit/glue/resource_loader_bridge.h"
@@ -122,6 +123,26 @@
       info->request_headers_text = request_line + request_headers.ToString();
       break;
     }
+    case net::NetLog::TYPE_HTTP_TRANSACTION_SPDY_SEND_REQUEST_HEADERS: {
+      scoped_ptr<Value> event_params(entry.ParametersToValue());
+      net::SpdyHeaderBlock request_headers;
+
+      if (!net::SpdyHeaderBlockFromNetLogParam(event_params.get(),
+                                               &request_headers)) {
+        NOTREACHED();
+      }
+
+      // We need to clear headers in case the same url_request is reused for
+      // several http requests (e.g. see http://crbug.com/80157).
+      info->request_headers.clear();
+
+      for (net::SpdyHeaderBlock::const_iterator it = request_headers.begin();
+           it != request_headers.end(); ++it) {
+        info->request_headers.push_back(std::make_pair(it->first, it->second));
+      }
+      info->request_headers_text = "";

[vsevik, 2012/09/06 17:32:45]

Can we treat request_headers_text better?

[vsevik, 2012/09/06 17:32:45]

Can we treat request_headers_text better?

[mmenke, 2012/09/06 17:45:31]

Note that SPDY doesn't have any request line - instead it breaks that
information into special headers (Which appear in request_headers).  There's no
code that strongly expects it to start with a request line?  I've gone ahead and
copied the behavior of HttpRequestHeaders::ToString.

+      break;
+    }
     case net::NetLog::TYPE_HTTP_TRANSACTION_READ_RESPONSE_HEADERS: {
       scoped_ptr<Value> event_params(entry.ParametersToValue());