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

Unified Diff: net/http/http_response_headers.cc

Issue 8669014: Fix a bug where redirect chain gets lost on process swap. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: latest Created 9 years 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
Index: net/http/http_response_headers.cc
diff --git a/net/http/http_response_headers.cc b/net/http/http_response_headers.cc
index f249db88e236a4ea4ddd03f0397e401a046f71ea..7b493157d454fefe2edd9fa6ac3310da39e1a2c8 100644
--- a/net/http/http_response_headers.cc
+++ b/net/http/http_response_headers.cc
@@ -353,8 +353,8 @@ void HttpResponseHeaders::RemoveHeaderWithValue(const std::string& name,
void HttpResponseHeaders::AddHeader(const std::string& header) {
CheckDoesNotHaveEmbededNulls(header);
- DCHECK_EQ('\0', raw_headers_[raw_headers_.size() - 2]);
- DCHECK_EQ('\0', raw_headers_[raw_headers_.size() - 1]);
+ //DCHECK_EQ('\0', raw_headers_[raw_headers_.size() - 2]);
+ //DCHECK_EQ('\0', raw_headers_[raw_headers_.size() - 1]);
Charlie Reis 2011/12/02 22:38:11 Needs cleanup. Why isn't this true anymore?
Matt Perry 2011/12/03 00:14:24 Oops, that was for an earlier bug that has been fi
// Don't copy the last null.
std::string new_raw_headers(raw_headers_, 0, raw_headers_.size() - 1);
new_raw_headers.append(header);
« content/renderer/render_view_impl.cc ('K') | « content/renderer/render_view_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698