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

Unified Diff: net/http/http_response_headers.h

Issue 10809011: Fix removal of headers (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 5 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
Index: net/http/http_response_headers.h
diff --git a/net/http/http_response_headers.h b/net/http/http_response_headers.h
index f6a270acae3ea1e3269718a466b8ca379196d9d5..8c462728b38e6b2e28a3f8871fcd59869b3bb754 100644
--- a/net/http/http_response_headers.h
+++ b/net/http/http_response_headers.h
@@ -65,9 +65,9 @@ class NET_EXPORT HttpResponseHeaders
// Removes all instances of a particular header.
void RemoveHeader(const std::string& name);
- // Removes a particular header. The header name is compared
+ // Removes a particular header line. The header name is compared
// case-insensitively.
- void RemoveHeaderWithValue(const std::string& name, const std::string& value);
+ void RemoveHeaderLine(const std::string& name, const std::string& values);
rvargas (doing something else) 2012/07/19 19:11:02 nit: I'm not sure about using plural for value...
battre 2012/07/23 13:31:11 Done.
// Adds a particular header. |header| has to be a single header without any
// EOL termination, just [<header-name>: <header-values>]

Powered by Google App Engine
This is Rietveld 408576698