Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(82)

Unified Diff: net/spdy/spdy_protocol.h

Issue 1357953002: Replace the existing SpdyHeaderBlock typedef with a class. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add NET_EXPORT to fix compile error on win_chromium_compile_dbg_ng. Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/spdy/spdy_network_transaction_unittest.cc ('k') | net/spdy/spdy_proxy_client_socket_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_protocol.h
diff --git a/net/spdy/spdy_protocol.h b/net/spdy/spdy_protocol.h
index 59985ede0b2a7c87aacb0209bf91cb346ede29da..c5d66c917d1362c73d819ac6dc2612399fc4557f 100644
--- a/net/spdy/spdy_protocol.h
+++ b/net/spdy/spdy_protocol.h
@@ -24,6 +24,7 @@
#include "net/base/net_export.h"
#include "net/spdy/spdy_alt_svc_wire_format.h"
#include "net/spdy/spdy_bitmasks.h"
+#include "net/spdy/spdy_header_block.h"
namespace net {
@@ -420,10 +421,6 @@ enum SpdyGoAwayStatus {
// number between 0 and 3.
typedef uint8 SpdyPriority;
-// A datastructure for holding a set of headers from a HEADERS, PUSH_PROMISE,
-// SYN_STREAM, or SYN_REPLY frame.
-typedef std::map<std::string, std::string> SpdyHeaderBlock;
-
typedef uint64 SpdyPingId;
typedef std::string SpdyProtocolId;
@@ -618,7 +615,7 @@ class NET_EXPORT_PRIVATE SpdyFrameWithHeaderBlockIR
header_block_ = header_block;
}
void SetHeader(base::StringPiece name, base::StringPiece value) {
- header_block_[name.as_string()] = value.as_string();
+ header_block_[name] = value;
}
SpdyHeaderBlock* mutable_header_block() { return &header_block_; }
« no previous file with comments | « net/spdy/spdy_network_transaction_unittest.cc ('k') | net/spdy/spdy_proxy_client_socket_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698