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

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

Issue 11931024: Removed static factories for data, ftp, file, and about jobs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Cleanup Created 7 years, 11 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
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 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 // This TransportSecurityState doesn't load or save any state. It's only 114 // This TransportSecurityState doesn't load or save any state. It's only
115 // used to enforce pinning for system requests and will only use built-in 115 // used to enforce pinning for system requests and will only use built-in
116 // pins. 116 // pins.
117 scoped_ptr<net::TransportSecurityState> transport_security_state; 117 scoped_ptr<net::TransportSecurityState> transport_security_state;
118 scoped_refptr<net::SSLConfigService> ssl_config_service; 118 scoped_refptr<net::SSLConfigService> ssl_config_service;
119 scoped_ptr<net::HttpAuthHandlerFactory> http_auth_handler_factory; 119 scoped_ptr<net::HttpAuthHandlerFactory> http_auth_handler_factory;
120 scoped_ptr<net::HttpServerProperties> http_server_properties; 120 scoped_ptr<net::HttpServerProperties> http_server_properties;
121 scoped_ptr<net::ProxyService> proxy_script_fetcher_proxy_service; 121 scoped_ptr<net::ProxyService> proxy_script_fetcher_proxy_service;
122 scoped_ptr<net::HttpTransactionFactory> 122 scoped_ptr<net::HttpTransactionFactory>
123 proxy_script_fetcher_http_transaction_factory; 123 proxy_script_fetcher_http_transaction_factory;
124 scoped_ptr<net::FtpTransactionFactory>
125 proxy_script_fetcher_ftp_transaction_factory;
126 scoped_ptr<net::URLRequestThrottlerManager> throttler_manager; 124 scoped_ptr<net::URLRequestThrottlerManager> throttler_manager;
127 scoped_ptr<net::URLSecurityManager> url_security_manager; 125 scoped_ptr<net::URLSecurityManager> url_security_manager;
128 // TODO(willchan): Remove proxy script fetcher context since it's not 126 // TODO(willchan): Remove proxy script fetcher context since it's not
129 // necessary now that I got rid of refcounting URLRequestContexts. 127 // necessary now that I got rid of refcounting URLRequestContexts.
130 // 128 //
131 // The first URLRequestContext is |system_url_request_context|. We introduce 129 // The first URLRequestContext is |system_url_request_context|. We introduce
132 // |proxy_script_fetcher_context| for the second context. It has a direct 130 // |proxy_script_fetcher_context| for the second context. It has a direct
133 // ProxyService, since we always directly connect to fetch the PAC script. 131 // ProxyService, since we always directly connect to fetch the PAC script.
134 scoped_ptr<net::URLRequestContext> proxy_script_fetcher_context; 132 scoped_ptr<net::URLRequestContext> proxy_script_fetcher_context;
135 scoped_ptr<net::ProxyService> system_proxy_service; 133 scoped_ptr<net::ProxyService> system_proxy_service;
136 scoped_ptr<net::HttpTransactionFactory> system_http_transaction_factory; 134 scoped_ptr<net::HttpTransactionFactory> system_http_transaction_factory;
137 scoped_ptr<net::FtpTransactionFactory> system_ftp_transaction_factory;
138 scoped_ptr<net::URLRequestContext> system_request_context; 135 scoped_ptr<net::URLRequestContext> system_request_context;
139 SystemRequestContextLeakChecker system_request_context_leak_checker; 136 SystemRequestContextLeakChecker system_request_context_leak_checker;
140 // |system_cookie_store| and |system_server_bound_cert_service| are shared 137 // |system_cookie_store| and |system_server_bound_cert_service| are shared
141 // between |proxy_script_fetcher_context| and |system_request_context|. 138 // between |proxy_script_fetcher_context| and |system_request_context|.
142 scoped_refptr<net::CookieStore> system_cookie_store; 139 scoped_refptr<net::CookieStore> system_cookie_store;
143 scoped_refptr<extensions::EventRouterForwarder> 140 scoped_refptr<extensions::EventRouterForwarder>
144 extension_event_router_forwarder; 141 extension_event_router_forwarder;
145 scoped_ptr<chrome_browser_net::HttpPipeliningCompatibilityClient> 142 scoped_ptr<chrome_browser_net::HttpPipeliningCompatibilityClient>
146 http_pipelining_compatibility_client; 143 http_pipelining_compatibility_client;
147 scoped_ptr<chrome_browser_net::LoadTimeStats> load_time_stats; 144 scoped_ptr<chrome_browser_net::LoadTimeStats> load_time_stats;
(...skipping 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
297 294
298 // True if SPDY is disabled by policy. 295 // True if SPDY is disabled by policy.
299 bool is_spdy_disabled_by_policy_; 296 bool is_spdy_disabled_by_policy_;
300 297
301 base::WeakPtrFactory<IOThread> weak_factory_; 298 base::WeakPtrFactory<IOThread> weak_factory_;
302 299
303 DISALLOW_COPY_AND_ASSIGN(IOThread); 300 DISALLOW_COPY_AND_ASSIGN(IOThread);
304 }; 301 };
305 302
306 #endif // CHROME_BROWSER_IO_THREAD_H_ 303 #endif // CHROME_BROWSER_IO_THREAD_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698