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

Unified Diff: net/url_request/url_request_context_builder.cc

Issue 10918279: Provide mutable members of UrlRequestContext via pure-virtual interface (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Disable test for chrome_frame_net_tests Created 8 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
« no previous file with comments | « net/url_request/url_request_context_builder.h ('k') | net/url_request/url_request_context_storage.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/url_request/url_request_context_builder.cc
diff --git a/net/url_request/url_request_context_builder.cc b/net/url_request/url_request_context_builder.cc
index 217337f19d783aebf6237121012e80bb0c51d678..f1cf1aaa0c061f838ce6ab1fd0955919219b3cfe 100644
--- a/net/url_request/url_request_context_builder.cc
+++ b/net/url_request/url_request_context_builder.cc
@@ -9,6 +9,7 @@
#include "base/basictypes.h"
#include "base/compiler_specific.h"
#include "base/logging.h"
+#include "base/string_util.h"
#include "base/threading/thread.h"
#include "base/thread_task_runner_handle.h"
#include "net/base/cert_verifier.h"
@@ -25,6 +26,7 @@
#include "net/http/http_network_session.h"
#include "net/http/http_server_properties_impl.h"
#include "net/proxy/proxy_service.h"
+#include "net/url_request/static_http_user_agent_settings.h"
#include "net/url_request/url_request_context.h"
#include "net/url_request/url_request_context_storage.h"
@@ -129,10 +131,6 @@ class BasicURLRequestContext : public URLRequestContext {
return &storage_;
}
- void set_user_agent(const std::string& user_agent) {
- user_agent_ = user_agent;
- }
-
void StartCacheThread() {
cache_thread_.StartWithOptions(
base::Thread::Options(MessageLoop::TYPE_IO, 0));
@@ -153,16 +151,10 @@ class BasicURLRequestContext : public URLRequestContext {
return file_thread_.message_loop();
}
- virtual const std::string& GetUserAgent(
- const GURL& /* url */) const OVERRIDE {
- return user_agent_;
- }
-
protected:
virtual ~BasicURLRequestContext() {}
private:
- std::string user_agent_;
base::Thread cache_thread_;
base::Thread file_thread_;
URLRequestContextStorage storage_;
@@ -203,7 +195,8 @@ URLRequestContext* URLRequestContextBuilder::Build() {
BasicURLRequestContext* context = new BasicURLRequestContext;
URLRequestContextStorage* storage = context->storage();
- context->set_user_agent(user_agent_);
+ storage->set_http_user_agent_settings(new StaticHttpUserAgentSettings(
+ accept_language_, accept_charset_, user_agent_));
if (!network_delegate_)
network_delegate_.reset(new BasicNetworkDelegate);
« no previous file with comments | « net/url_request/url_request_context_builder.h ('k') | net/url_request/url_request_context_storage.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698