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

Unified Diff: chrome/browser/safe_browsing/safe_browsing_service.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 | « chrome/browser/profiles/profile_io_data.cc ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/safe_browsing/safe_browsing_service.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_service.cc b/chrome/browser/safe_browsing/safe_browsing_service.cc
index 45a5a867ec19b06aad9761bea844f751b115540a..3b112d2c9bc2b01362f91b5b5506c80bc6be9aec 100644
--- a/chrome/browser/safe_browsing/safe_browsing_service.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_service.cc
@@ -100,22 +100,6 @@ FilePath CookieFilePath() {
} // namespace
-// Custom URLRequestContext used by SafeBrowsing requests, which are not
-// associated with a particular profile. We need to use a subclass of
-// URLRequestContext in order to provide the correct User-Agent.
-class SafeBrowsingURLRequestContext : public net::URLRequestContext {
- public:
- virtual const std::string& GetUserAgent(
- const GURL& url) const OVERRIDE {
- return content::GetUserAgent(url);
- }
-
- private:
- virtual ~SafeBrowsingURLRequestContext() {}
-
- base::debug::LeakTracker<SafeBrowsingURLRequestContext> leak_tracker_;
-};
-
class SafeBrowsingURLRequestContextGetter
: public net::URLRequestContextGetter {
public:
@@ -624,11 +608,12 @@ void SafeBrowsingService::InitURLRequestContextOnIOThread(
new SQLitePersistentCookieStore(CookieFilePath(), false, NULL),
NULL);
- url_request_context_.reset(new SafeBrowsingURLRequestContext);
+ url_request_context_.reset(new net::URLRequestContext);
// |system_url_request_context_getter| may be NULL during tests.
- if (system_url_request_context_getter)
+ if (system_url_request_context_getter) {
url_request_context_->CopyFrom(
system_url_request_context_getter->GetURLRequestContext());
+ }
url_request_context_->set_cookie_store(cookie_store);
}
« no previous file with comments | « chrome/browser/profiles/profile_io_data.cc ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698