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

Unified Diff: sync/internal_api/http_bridge.cc

Issue 83433008: Cleanup: Remove legacy accessors from URLRequestContext. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 1 month 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: sync/internal_api/http_bridge.cc
diff --git a/sync/internal_api/http_bridge.cc b/sync/internal_api/http_bridge.cc
index 5827c9d77b50c0dbc9ba61a92152241c6b44cd21..136bd0c266245742694da974c68a7b3b38a947da 100644
--- a/sync/internal_api/http_bridge.cc
+++ b/sync/internal_api/http_bridge.cc
@@ -151,9 +151,13 @@ HttpBridge::RequestContext::RequestContext(
// should be tied to whatever the sync servers expect (if anything). These
// fields should probably just be settable by sync backend; though we should
// figure out if we need to give the user explicit control over policies etc.
+ std::string accepted_language_list;
+ if (baseline_context->http_user_agent_settings()) {
+ accepted_language_list =
+ baseline_context->http_user_agent_settings()->GetAcceptLanguage();
+ }
http_user_agent_settings_.reset(new net::StaticHttpUserAgentSettings(
- baseline_context->GetAcceptLanguage(),
- user_agent));
+ accepted_language_list, user_agent));
pauljensen 2013/12/02 15:24:42 I think these two arguments should be on seperate
Lei Zhang 2013/12/11 03:32:24 Done.
set_http_user_agent_settings(http_user_agent_settings_.get());
set_net_log(baseline_context->net_log());

Powered by Google App Engine
This is Rietveld 408576698