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

Unified Diff: net/quic/spdy_utils.cc

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/net.gypi ('k') | net/quic/test_tools/quic_test_packet_maker.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/spdy_utils.cc
diff --git a/net/quic/spdy_utils.cc b/net/quic/spdy_utils.cc
index c03d5887cfe50f00b0c706b375c3304a52083b29..0ed9e9df90e001f267fa776750bfb2e992bd4000 100644
--- a/net/quic/spdy_utils.cc
+++ b/net/quic/spdy_utils.cc
@@ -27,9 +27,10 @@ SpdyHeaderBlock SpdyUtils::ConvertSpdy3ResponseHeadersToSpdy4(
SpdyHeaderBlock response_headers) {
// SPDY/4 headers include neither the version field nor the response details.
response_headers.erase(":version");
- size_t end_of_code = response_headers[":status"].find(' ');
+ StringPiece status_value = response_headers[":status"];
+ size_t end_of_code = status_value.find(' ');
if (end_of_code != string::npos) {
- response_headers[":status"].erase(end_of_code);
+ response_headers[":status"] = status_value.substr(0, end_of_code);
}
return response_headers;
}
« no previous file with comments | « net/net.gypi ('k') | net/quic/test_tools/quic_test_packet_maker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698