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

Unified Diff: net/url_request/url_request_http_job.cc

Issue 12463021: Stop sending Accept-Charset HTTP header. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix comment Created 7 years, 9 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/url_request/url_request_http_job.cc
diff --git a/net/url_request/url_request_http_job.cc b/net/url_request/url_request_http_job.cc
index 8dc1507b1f9f3bb64f95b8a4de6a89ee8b4a404d..29d46e0da9220e21b3e7f5a5ad90d20f909fa16f 100644
--- a/net/url_request/url_request_http_job.cc
+++ b/net/url_request/url_request_http_job.cc
@@ -482,8 +482,8 @@ void URLRequestHttpJob::AddExtraHeaders() {
}
if (http_user_agent_settings_) {
- // Only add default Accept-Language and Accept-Charset if the request
- // didn't have them specified.
+ // Only add default Accept-Language if the request didn't have it
+ // specified.
std::string accept_language =
http_user_agent_settings_->GetAcceptLanguage();
if (!accept_language.empty()) {
@@ -491,12 +491,6 @@ void URLRequestHttpJob::AddExtraHeaders() {
HttpRequestHeaders::kAcceptLanguage,
accept_language);
}
- std::string accept_charset = http_user_agent_settings_->GetAcceptCharset();
- if (!accept_charset.empty()) {
- request_info_.extra_headers.SetHeaderIfMissing(
- HttpRequestHeaders::kAcceptCharset,
- accept_charset);
- }
}
}

Powered by Google App Engine
This is Rietveld 408576698