summaryrefslogtreecommitdiffstats
path: root/chrome/service/net
Commit message (Collapse)AuthorAgeFilesLines
* [SPDY] Use WeakPtr<HttpServerProperties> instead of raw pointersakalin@chromium.org2013-07-181-1/+3
| | | | | | | | | | | | | | | This will let us better track down what is causing SpdySessions to be accessing them after they're destroyed, since we'll have crash reports instead of just the SyzyASAN reports. Also use scoped_ptr<HttpServerProperties> when appropriate. BUG=236451 TBR=ajwong@chromium.org, rtenneti@chromium.org, simonjam@chromium.org, wez@chromium.org Review URL: https://codereview.chromium.org/19731002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212466 0039d316-1c4b-4281-b951-d872f2087c98
* Give more request types a TransportSecurityState.palmer@chromium.org2013-06-131-0/+1
| | | | | | | | | | | | DCHECK on NULL TransportSecurityState, as a precursor to a real CHECK. It should be an error to try to connect with an SSL client socket without having a live TSS. BUG=246724 Review URL: https://chromiumcodereview.appspot.com/16501002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@206013 0039d316-1c4b-4281-b951-d872f2087c98
* Call scoped_refptr<T>::get() rather than relying on implicit "operator T*"rsleevi@chromium.org2013-06-111-4/+3
| | | | | | | | | | | | | | | | | | | This upates calls to bound temporary objects to also use get(). While it has the same semantic equivalence to the existing code, this generally represents a dangerous pattern - indeed, part of the whole motivation for this change is to make this anti-pattern very visible to authors. This change simply updates all of the call sites, to allow the "operator T*" to be removed and preventing new instances. The existing instances will then be reviewed for "suspicious" changes and updated to use/pass scoped_refptr<T> rather than T*, as appropriate. BUG=110610 TBR=darin Review URL: https://codereview.chromium.org/15984016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205560 0039d316-1c4b-4281-b951-d872f2087c98
* Use a direct include of strings headers in chrome/service/.avi@chromium.org2013-06-111-1/+1
| | | | | | | | | | BUG=247723 TEST=none TBR=ben@chromium.org Review URL: https://chromiumcodereview.appspot.com/16752002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205411 0039d316-1c4b-4281-b951-d872f2087c98
* Update chrome/ to use scoped_refptr<T>::get() rather than implicit "operator T*"rsleevi@chromium.org2013-06-021-4/+2
| | | | | | | | | | | Linux fixes, Part 1 of N BUG=110610 TBR=darin Review URL: https://chromiumcodereview.appspot.com/15836003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@203632 0039d316-1c4b-4281-b951-d872f2087c98
* chrome: Update the include paths of base/message_loop_proxy.h to its new ↵tfarina@chromium.org2013-05-101-1/+1
| | | | | | | | | | | | | | location. message_loop_proxy.h was moved into base/message_loop/ in r191422 - https://codereview.chromium.org/13333003 BUG=None TBR=darin@chromium.org,brettw@chromium.org Review URL: https://chromiumcodereview.appspot.com/14979003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@199389 0039d316-1c4b-4281-b951-d872f2087c98
* Removed static factories for data, ftp, file, and about jobs.pauljensen@chromium.org2013-05-081-3/+0
| | | | | | | | | | | | | | | | | Instead add corresponding ProtocolHandlers as needed. Remove URLRequestContext members used by these static factories. Bake FtpAuthCache into FtpProtocolHandler as it was already unique per FtpProtocolHandler. This is a revived version of http://crrev.com/10836206 BUG=142945 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=188912 Review URL: https://chromiumcodereview.appspot.com/11931024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198915 0039d316-1c4b-4281-b951-d872f2087c98
* chrome: Remove the remaining use of ALLOW_THIS_IN_INITIALIZER_LIST.tfarina@chromium.org2013-05-011-1/+1
| | | | | | | | | | | | It's no longer providing value as the MSVC warning is disabled during compilation. Refer to bug for details. BUG=234765 TBR=ben@chromium.org Review URL: https://codereview.chromium.org/14712004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@197552 0039d316-1c4b-4281-b951-d872f2087c98
* net: extract net/cert out of net/basephajdan.jr@chromium.org2013-03-291-1/+1
| | | | | | | | | | | | This introduces the following dependency of net/base on things outside: net/base/openssl_client_key_store.cc:#include "net/cert/x509_certificate.h" BUG=70818 Review URL: https://codereview.chromium.org/13006020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@191450 0039d316-1c4b-4281-b951-d872f2087c98
* net: move host_resolver files from net/base to net/dnsphajdan.jr@chromium.org2013-03-212-2/+2
| | | | | | | | BUG=70818 Review URL: https://codereview.chromium.org/12518036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189485 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 188912 "Removed static factories for data, ftp, file, and..."kbr@chromium.org2013-03-191-0/+3
| | | | | | | | | | | | | | | | | | | | | Broke layout tests userscripts/user-script-plugin-document.html and plugins/plugin-document-back-forward.html on all platforms. > Removed static factories for data, ftp, file, and about jobs. > Instead add corresponding ProtocolHandlers as needed. > Remove URLRequestContext members used by these static > factories. Bake FtpAuthCache into FtpProtocolHandler as it > was already unique per FtpProtocolHandler. > This is a revived version of http://crrev.com/10836206 > > BUG=142945 > > > Review URL: https://chromiumcodereview.appspot.com/11931024 TBR=pauljensen@chromium.org Review URL: https://codereview.chromium.org/12605011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@188927 0039d316-1c4b-4281-b951-d872f2087c98
* Removed static factories for data, ftp, file, and about jobs.pauljensen@chromium.org2013-03-191-3/+0
| | | | | | | | | | | | | | | Instead add corresponding ProtocolHandlers as needed. Remove URLRequestContext members used by these static factories. Bake FtpAuthCache into FtpProtocolHandler as it was already unique per FtpProtocolHandler. This is a revived version of http://crrev.com/10836206 BUG=142945 Review URL: https://chromiumcodereview.appspot.com/11931024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@188912 0039d316-1c4b-4281-b951-d872f2087c98
* Stop sending Accept-Charset HTTP header as its not relevant anymore. Remove ↵pauljensen@chromium.org2013-03-151-1/+1
| | | | | | | | | | it from HttpUserAgentSettings as well as this is no longer needed. BUG=112804 Review URL: https://chromiumcodereview.appspot.com/12463021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@188237 0039d316-1c4b-4281-b951-d872f2087c98
* net: split net/ssl out of net/basephajdan.jr@chromium.org2013-03-142-2/+2
| | | | | | | | | | | | | | | Also moves transport_security_state files to net/http. This change also updates the callers. R=wtc TBR=rsleevi,lambroslambrou,tony BUG=70818 Review URL: https://codereview.chromium.org/12680003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@188119 0039d316-1c4b-4281-b951-d872f2087c98
* Provide mutable members of UrlRequestContext via pure-virtual interface. ↵pauljensen@chromium.org2012-11-072-16/+4
| | | | | | | | | | Create a pure-virtual interface called HttpUserAgentSettings that provides access to the Accept-Language, Accept-Charset, and User-Agent HTTP headers. Each UrlRequestContext should have a HttpUserAgentSettings implementation attached via set_http_user_agent_settings(). BUG=146596 Review URL: https://chromiumcodereview.appspot.com/10918279 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@166425 0039d316-1c4b-4281-b951-d872f2087c98
* [net] Change factory methods for HostResolver and HostCache to return a ↵szym@chromium.org2012-10-221-4/+1
| | | | | | | | | | | | scoped_ptr. Move HostResolver factory methods to host_resolver.cc. This also fixes a double-free in ShellURLRequestContextGetter. Review URL: https://chromiumcodereview.appspot.com/10831277 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163402 0039d316-1c4b-4281-b951-d872f2087c98
* Bring back service process throttler.scottbyer@chromium.org2012-09-051-0/+2
| | | | | | | | | | | | | | Somewhere along the way, the service process lost it's throttler. Under certain conditions (network going away) that meant that the service process could spin trying to access something, eating up CPU. BUG=140019 TEST=(Mac) Enable cloud print, disable network. Before this change, after 30 minutes, the Chromium Helper process would be >100% CPU in Activity Monitor. After this change, should be near nothing. Review URL: https://chromiumcodereview.appspot.com/10919102 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155050 0039d316-1c4b-4281-b951-d872f2087c98
* Added TransportSecurityState to URLRequestContextStorage used in cloud print ↵vitalybuka@chromium.org2012-08-131-0/+1
| | | | | | | | | | connector. BUG=141932 Review URL: https://chromiumcodereview.appspot.com/10854122 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151331 0039d316-1c4b-4281-b951-d872f2087c98
* Remove #pragma once from chrome/serviceajwong@chromium.org2012-07-111-1/+0
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10693138 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146073 0039d316-1c4b-4281-b951-d872f2087c98
* Replace GetIOMessageLoopProxy() with GetNetworkTaskRunner() in ↵sergeyu@chromium.org2012-06-162-7/+7
| | | | | | | | | | | | UrlRequestContextGetter. Also updated all code that depends on that methods. TBR=mnissler@chromium.org Review URL: https://chromiumcodereview.appspot.com/10539148 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142560 0039d316-1c4b-4281-b951-d872f2087c98
* Use SingleThreadTaskRunner instead of MessageLoopProxy in ProxyConfigService.sergeyu@chromium.org2012-05-241-1/+1
| | | | | | | | MessageLoopProxy is not usable in plugin, and so it's not possible to use ProxyConfigService in plugins if it uses MessageLoopProxy. Review URL: https://chromiumcodereview.appspot.com/10414061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138751 0039d316-1c4b-4281-b951-d872f2087c98
* Stop refcounting URLRequestContext.willchan@chromium.org2012-05-112-8/+7
| | | | | | | | | | | | While doing so, fix a few issues with the code like ordering of URLRequestContext to ensure correct destruction order. Also fix const correctness in some places. BUG=58859 TEST=none TBR=willchan Review URL: https://chromiumcodereview.appspot.com/10299002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136497 0039d316-1c4b-4281-b951-d872f2087c98
* Make CertVerifier a pure virtual interface.rsleevi@chromium.org2012-03-211-1/+1
| | | | | | | | | | | | | The existing CertVerifier implementation has been renamed to MultiThreadedCertVerifier, consistent with ProxyResolver naming. This is patch 1 of N for http://crbug.com/114343 BUG=114343 TEST=Compiles and existing unittests pass. Review URL: https://chromiumcodereview.appspot.com/9476035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@128090 0039d316-1c4b-4281-b951-d872f2087c98
* Update net/base/cookie_*.h includes to net/cookies/cookie_*.h .erikwright@chromium.org2012-03-152-4/+4
| | | | | | | | | | | | Once complete, forwarding headers will be removed from net/base/ . TEST=none R=willchan Review URL: http://codereview.chromium.org/9703062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@126999 0039d316-1c4b-4281-b951-d872f2087c98
* net: replace DnsRRResoler with TransportSecurityState in plumbing.agl@chromium.org2011-12-071-3/+0
| | | | | | | | | | | | | | | | | | The DnsRRResoler has served its time and I don't have any further plans for it. The TransportSecurityState will be used (in a future CL) to enforce public key pinning at certificate verification time. Currently we only enforce it in url_request_http_job.cc which means that we've already sent the HTTP request (inc cookies) by the time we catch the problem. This was expeditious while we fixed some other issues, but it's time to fix it. BUG=none TEST=compiles Review URL: http://codereview.chromium.org/8692012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113410 0039d316-1c4b-4281-b951-d872f2087c98
* Add OVERRIDE to chrome/service/.avi@chromium.org2011-11-191-3/+4
| | | | | | | | | BUG=104314 TEST=no change Review URL: http://codereview.chromium.org/8549032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110808 0039d316-1c4b-4281-b951-d872f2087c98
* Migrate HttpAlternateProtocols to HttpServerPropertiesImpl.willchan@chromium.org2011-10-101-0/+3
| | | | | | | | | | | | | Hooks in HttpServerPropertiesManager also. No persistence done yet. This is all plumbing. Also require HttpServerProperties in HttpNetworkSession::Params. BUG=98472 TEST=none Review URL: http://codereview.chromium.org/8211003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104696 0039d316-1c4b-4281-b951-d872f2087c98
* Revert of 85854 which is a revert of 85852 - Add a command line optionrtenneti@chromium.org2011-05-191-0/+1
| | | | | | | | | | | | | | | ("host-resolver-retry-attempts") to specify the number of retry attempts to resolve host. Relanding CL http://codereview.chromium.org/7011044/ BUG=82580 TEST=host resolver unit tests TBR=eroman Review URL: http://codereview.chromium.org/7011044 Review URL: http://codereview.chromium.org/7029050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@85871 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 85852 - Add a command line option ("host-resolver-retry-attempts")rtenneti@chromium.org2011-05-191-1/+0
| | | | | | | | | | | | | | | to specify the number of retry attempts to resolve host. BUG=82580 TEST=host resolver unit tests R=eroman Review URL: http://codereview.chromium.org/7011044 TBR=rtenneti@chromium.org Review URL: http://codereview.chromium.org/7044027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@85854 0039d316-1c4b-4281-b951-d872f2087c98
* Add a command line option ("host-resolver-retry-attempts")rtenneti@chromium.org2011-05-191-0/+1
| | | | | | | | | | | | to specify the number of retry attempts to resolve host. BUG=82580 TEST=host resolver unit tests R=eroman Review URL: http://codereview.chromium.org/7011044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@85852 0039d316-1c4b-4281-b951-d872f2087c98
* Get rid of net::CookiePolicy, now that all code that uses it (except ↵jam@chromium.org2011-05-122-2/+0
| | | | | | | | | WebSocketJob, which appears to be unused and which I updated in this cl) is switched over to use ContentBrowserClient. BUG=76793 Review URL: http://codereview.chromium.org/6973011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@85136 0039d316-1c4b-4281-b951-d872f2087c98
* iwyu: Include stringprintf.h where appropriate, part 2.jhawkins@chromium.org2011-05-111-1/+1
| | | | | | | | | | | BUG=82098 TEST=none R=thakis@chromium.org Review URL: http://codereview.chromium.org/7004007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@85048 0039d316-1c4b-4281-b951-d872f2087c98
* Fix Linux proxy.scottbyer@chromium.org2011-04-282-34/+24
| | | | | | | | | | | | | | | Somewhere along the way, the proxy service creation code got more strict and the fragile earlier fix broke. This is much more correct and consistent across platforms. Create the proxy config service on the UI thread, create the proxy service itself on the IO thread. BUG=none TEST=start up the linux proxy, it works again. Check windows, still works. Check Mac, still works. Review URL: http://codereview.chromium.org/6905046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83350 0039d316-1c4b-4281-b951-d872f2087c98
* Stop refcounting ProxyService.willchan@chromium.org2011-04-272-29/+32
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/6873096 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83222 0039d316-1c4b-4281-b951-d872f2087c98
* Undo raw DNS resolver changes, this will be implemented differently, on ↵agayev@chromium.org2011-04-271-1/+1
| | | | | | | | | | | IO-thread. BUG=60149 TEST=None Review URL: http://codereview.chromium.org/6893007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83177 0039d316-1c4b-4281-b951-d872f2087c98
* Moved URLRequestContextGetter to net/ so it can be used by projects such as ↵sanjeevr@chromium.org2011-03-311-2/+2
| | | | | | | | | | jingle. BUG=None TEST=Build. Review URL: http://codereview.chromium.org/6778025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80033 0039d316-1c4b-4281-b951-d872f2087c98
* Added the HttpAuthHandlerFactory pointer to the ↵sanjeevr@chromium.org2011-03-101-0/+2
| | | | | | | | | | | | net::HttpNetworkSession::Params structure. This fixes the service process crash as seen here: http://crash/reportdetail?reportid=2e6994509abbee7b BUG=75692 TEST=Cloud Print proxy for regressions, we should not see that crash any more. Review URL: http://codereview.chromium.org/6672002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77708 0039d316-1c4b-4281-b951-d872f2087c98
* Service process crash on Windows.scottbyer@chromium.org2011-03-051-4/+4
| | | | | | | | | | | | A crash not caught by the service process browser tests, revert the timeline of creating the proxy service on Windows to what it was previously. BUG=74992 TEST=enable cloud print proxy on Windows, service doesn't crash withing 5 minutes. Review URL: http://codereview.chromium.org/6626038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77017 0039d316-1c4b-4281-b951-d872f2087c98
* Service process fixes for Linux.scottbyer@chromium.org2011-03-032-15/+35
| | | | | | | | | | | | | | | | | | Linux requires that the proxy_config_service be created on the I/O thread. Fortunately, ServiceURLRequestContextGetter was already thread safe ref counted, so we lift creation of that and the (also, fortunately) thread safe ref counted net::ProxyService up to service process initialization time. That created proxy service now gets passed in to the dynamic creation of the ServiceURLRequestContexts. Should have no effect on other platforms. BUG=74226 TEST=Start Chromium on Linux with --enable-cloud-print-proxy, sign in, and examine ~/.config/chromium/Service State. With the fix, Service State properly contains the needed tokens to properly re-launch and be a proxy. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=76612 Review URL: http://codereview.chromium.org/6602041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76760 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 76612 - Service process fixes for Linux.scottbyer@chromium.org2011-03-022-35/+15
| | | | | | | | | | | | | | | | | | | Linux requires that the proxy_config_service be created on the I/O thread. Fortunately, ServiceURLRequestContextGetter was already thread safe ref counted, so we lift creation of that and the (also, fortunately) thread safe ref counted net::ProxyService up to service process initialization time. That created proxy service now gets passed in to the dynamic creation of the ServiceURLRequestContexts. Should have no effect on other platforms. BUG=74226 TEST=Start Chromium on Linux with --enable-cloud-print-proxy, sign in, and examine ~/.config/chromium/Service State. With the fix, Service State properly contains the needed tokens to properly re-launch and be a proxy. Review URL: http://codereview.chromium.org/6602041 TBR=scottbyer@chromium.org Review URL: http://codereview.chromium.org/6606022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76638 0039d316-1c4b-4281-b951-d872f2087c98
* Service process fixes for Linux.scottbyer@chromium.org2011-03-022-15/+35
| | | | | | | | | | | | | | | | Linux requires that the proxy_config_service be created on the I/O thread. Fortunately, ServiceURLRequestContextGetter was already thread safe ref counted, so we lift creation of that and the (also, fortunately) thread safe ref counted net::ProxyService up to service process initialization time. That created proxy service now gets passed in to the dynamic creation of the ServiceURLRequestContexts. Should have no effect on other platforms. BUG=74226 TEST=Start Chromium on Linux with --enable-cloud-print-proxy, sign in, and examine ~/.config/chromium/Service State. With the fix, Service State properly contains the needed tokens to properly re-launch and be a proxy. Review URL: http://codereview.chromium.org/6602041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76612 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r74842willchan@chromium.org2011-02-162-30/+28
| | | | | | | | | | | Merged correctly this time. BUG=none TEST=none Review URL: http://codereview.chromium.org/6523067 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75165 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 75143 - Reland r74842willchan@chromium.org2011-02-162-28/+30
| | | | | | | | | | | | | | The bug was that the ProfilePolicyContext was getting initialized before the ProfileImplIOData::Handle was initialized. BUG=67237 TEST=none Review URL: http://codereview.chromium.org/6526029 TBR=willchan@chromium.org Review URL: http://codereview.chromium.org/6480120 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75152 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r74842willchan@chromium.org2011-02-162-30/+28
| | | | | | | | | | | The bug was that the ProfilePolicyContext was getting initialized before the ProfileImplIOData::Handle was initialized. BUG=67237 TEST=none Review URL: http://codereview.chromium.org/6526029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75143 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 74842 - It seems to have broken the ChromeOS "PFQ bot"?willchan@chromium.org2011-02-142-28/+30
| | | | | | | | | | | | | | | | r74561 added a DCHECK to make sure users didn't try to access the ChromeURLRequestContextGetter from the Profile, since the Profile should only be read on the UI thread. ChromeOS apparently tried to access it from another thread, and therefore hit the new DCHECK. I'm relanding without the DCHECK. I'm also eagerly initializing the off the record context getter to prevent the ChromeOS race. ChromeOS should fix that code so the eager initialization isn't necessary. BUG=none TEST=none Review URL: http://codereview.chromium.org/6484041 TBR=willchan@chromium.org Review URL: http://codereview.chromium.org/6517021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74848 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r74728 which reverted r74561 for exposing a ChromeOS bug.willchan@chromium.org2011-02-142-30/+28
| | | | | | | | | | | | | r74561 added a DCHECK to make sure users didn't try to access the ChromeURLRequestContextGetter from the Profile, since the Profile should only be read on the UI thread. ChromeOS apparently tried to access it from another thread, and therefore hit the new DCHECK. I'm relanding without the DCHECK. I'm also eagerly initializing the off the record context getter to prevent the ChromeOS race. ChromeOS should fix that code so the eager initialization isn't necessary. BUG=none TEST=none Review URL: http://codereview.chromium.org/6484041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74842 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r74632 "Reland r74561 after fixing ChromeOS build breakage."oshima@google.com2011-02-122-28/+30
| | | | | | | | | | | | This reverts commit f19e6d524e086ed8182a475680e931a9a870d2a8. TBR=willchan@chromium.org BUG=none TEST=chromeos pfq bot should cycle green Review URL: http://codereview.chromium.org/6512009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74728 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r74561 after fixing ChromeOS build breakage.willchan@chromium.org2011-02-112-30/+28
| | | | | | | | | | | | | | | | | Introduce OffTheRecordProfileIOData and ProfileImplIOData. They both inherit from ProfileIOData. The former is for the off the record (incognito) profile. The latter is for the normal ProfileImpl profile. All of the IO related Profile objects are now initialized at the same time, in the subtype implementations of ProfileIOData::LazyInitializeInternal(). I also took this opportunity to clean URLRequestContext up so it is a class and keeps its member variables private. This required touching a fair number of files. TODO: Remove lots of the refcounting of member variables, since they can now be owned by ProfileIOData. BUG=67237 TEST=none Review URL: http://codereview.chromium.org/6500002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74632 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 74561 for breaking the ChromeOS buildwillchan@chromium.org2011-02-112-28/+30
| | | | | | | | | | | | | | | | | | | | Introduce OffTheRecordProfileIOData and ProfileImplIOData. They both inherit from ProfileIOData. The former is for the off the record (incognito) profile. The latter is for the normal ProfileImpl profile. All of the IO related Profile objects are now initialized at the same time, in the subtype implementations of ProfileIOData::LazyInitializeInternal(). I also took this opportunity to clean URLRequestContext up so it is a class and keeps its member variables private. This required touching a fair number of files. TODO: Remove lots of the refcounting of member variables, since they can now be owned by ProfileIOData. BUG=67237 TEST=none Review URL: http://codereview.chromium.org/6286133 TBR=willchan@chromium.org Review URL: http://codereview.chromium.org/6489020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74564 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce OffTheRecordProfileIOData and ProfileImplIOData.willchan@chromium.org2011-02-112-30/+28
| | | | | | | | | | | | | | | They both inherit from ProfileIOData. The former is for the off the record (incognito) profile. The latter is for the normal ProfileImpl profile. All of the IO related Profile objects are now initialized at the same time, in the subtype implementations of ProfileIOData::LazyInitializeInternal(). I also took this opportunity to clean URLRequestContext up so it is a class and keeps its member variables private. This required touching a fair number of files. TODO: Remove lots of the refcounting of member variables, since they can now be owned by ProfileIOData. BUG=67237 TEST=none Review URL: http://codereview.chromium.org/6286133 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74561 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-03-16 08:11:45 +0000