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

Side by Side Diff: chrome/browser/io_thread.h

Issue 10203002: Make URLRequestThrottlerManager a member of URLRequestContext. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Respond to review comments. Created 8 years, 7 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | chrome/browser/io_thread.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_IO_THREAD_H_ 5 #ifndef CHROME_BROWSER_IO_THREAD_H_
6 #define CHROME_BROWSER_IO_THREAD_H_ 6 #define CHROME_BROWSER_IO_THREAD_H_
7 #pragma once 7 #pragma once
8 8
9 #include <string> 9 #include <string>
10 10
(...skipping 28 matching lines...) Expand all
39 class HttpTransactionFactory; 39 class HttpTransactionFactory;
40 class NetworkDelegate; 40 class NetworkDelegate;
41 class ServerBoundCertService; 41 class ServerBoundCertService;
42 class ProxyConfigService; 42 class ProxyConfigService;
43 class ProxyService; 43 class ProxyService;
44 class SdchManager; 44 class SdchManager;
45 class SSLConfigService; 45 class SSLConfigService;
46 class TransportSecurityState; 46 class TransportSecurityState;
47 class URLRequestContext; 47 class URLRequestContext;
48 class URLRequestContextGetter; 48 class URLRequestContextGetter;
49 class URLRequestThrottlerManager;
49 class URLSecurityManager; 50 class URLSecurityManager;
50 } // namespace net 51 } // namespace net
51 52
52 // Contains state associated with, initialized and cleaned up on, and 53 // Contains state associated with, initialized and cleaned up on, and
53 // primarily used on, the IO thread. 54 // primarily used on, the IO thread.
54 // 55 //
55 // If you are looking to interact with the IO thread (e.g. post tasks 56 // If you are looking to interact with the IO thread (e.g. post tasks
56 // to it or check if it is the current thread), see 57 // to it or check if it is the current thread), see
57 // content::BrowserThread. 58 // content::BrowserThread.
58 class IOThread : public content::BrowserThreadDelegate { 59 class IOThread : public content::BrowserThreadDelegate {
(...skipping 20 matching lines...) Expand all
79 // pins. 80 // pins.
80 scoped_ptr<net::TransportSecurityState> transport_security_state; 81 scoped_ptr<net::TransportSecurityState> transport_security_state;
81 scoped_refptr<net::SSLConfigService> ssl_config_service; 82 scoped_refptr<net::SSLConfigService> ssl_config_service;
82 scoped_ptr<net::HttpAuthHandlerFactory> http_auth_handler_factory; 83 scoped_ptr<net::HttpAuthHandlerFactory> http_auth_handler_factory;
83 scoped_ptr<net::HttpServerProperties> http_server_properties; 84 scoped_ptr<net::HttpServerProperties> http_server_properties;
84 scoped_ptr<net::ProxyService> proxy_script_fetcher_proxy_service; 85 scoped_ptr<net::ProxyService> proxy_script_fetcher_proxy_service;
85 scoped_ptr<net::HttpTransactionFactory> 86 scoped_ptr<net::HttpTransactionFactory>
86 proxy_script_fetcher_http_transaction_factory; 87 proxy_script_fetcher_http_transaction_factory;
87 scoped_ptr<net::FtpTransactionFactory> 88 scoped_ptr<net::FtpTransactionFactory>
88 proxy_script_fetcher_ftp_transaction_factory; 89 proxy_script_fetcher_ftp_transaction_factory;
90 scoped_ptr<net::URLRequestThrottlerManager> throttler_manager;
89 scoped_ptr<net::URLSecurityManager> url_security_manager; 91 scoped_ptr<net::URLSecurityManager> url_security_manager;
90 // We use a separate URLRequestContext for PAC fetches, in order to break 92 // We use a separate URLRequestContext for PAC fetches, in order to break
91 // the reference cycle: 93 // the reference cycle:
92 // URLRequestContext=>PAC fetch=>URLRequest=>URLRequestContext. 94 // URLRequestContext=>PAC fetch=>URLRequest=>URLRequestContext.
93 // The first URLRequestContext is |system_url_request_context|. We introduce 95 // The first URLRequestContext is |system_url_request_context|. We introduce
94 // |proxy_script_fetcher_context| for the second context. It has a direct 96 // |proxy_script_fetcher_context| for the second context. It has a direct
95 // ProxyService, since we always directly connect to fetch the PAC script. 97 // ProxyService, since we always directly connect to fetch the PAC script.
96 scoped_refptr<net::URLRequestContext> proxy_script_fetcher_context; 98 scoped_refptr<net::URLRequestContext> proxy_script_fetcher_context;
97 scoped_ptr<net::ProxyService> system_proxy_service; 99 scoped_ptr<net::ProxyService> system_proxy_service;
98 scoped_ptr<net::HttpTransactionFactory> system_http_transaction_factory; 100 scoped_ptr<net::HttpTransactionFactory> system_http_transaction_factory;
(...skipping 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
209 system_url_request_context_getter_; 211 system_url_request_context_getter_;
210 212
211 net::SdchManager* sdch_manager_; 213 net::SdchManager* sdch_manager_;
212 214
213 base::WeakPtrFactory<IOThread> weak_factory_; 215 base::WeakPtrFactory<IOThread> weak_factory_;
214 216
215 DISALLOW_COPY_AND_ASSIGN(IOThread); 217 DISALLOW_COPY_AND_ASSIGN(IOThread);
216 }; 218 };
217 219
218 #endif // CHROME_BROWSER_IO_THREAD_H_ 220 #endif // CHROME_BROWSER_IO_THREAD_H_
OLDNEW
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | chrome/browser/io_thread.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698