summaryrefslogtreecommitdiffstats
path: root/chrome/common/net
Commit message (Collapse)AuthorAgeFilesLines
* net: Remove typedef net::URLRequestJob URLRequestJob;tfarina@chromium.org2010-12-112-3/+3
| | | | | | | | | BUG=64263 TEST=compiled locally, trybots Review URL: http://codereview.chromium.org/5607004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68935 0039d316-1c4b-4281-b951-d872f2087c98
* Start deinlining non-empty virtual methods. (This will be automatically checkederg@google.com2010-12-092-7/+14
| | | | | | | | | | | for in the future.) BUG=none TEST=compiles Review URL: http://codereview.chromium.org/5574006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68746 0039d316-1c4b-4281-b951-d872f2087c98
* Add the "virtual" keyword on method overrides that are missing it.erg@google.com2010-12-081-6/+6
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/5648004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68606 0039d316-1c4b-4281-b951-d872f2087c98
* Stop using adding refcounts to some URLRequestJob subtypes.willchan@chromium.org2010-12-032-3/+10
| | | | | | | | | | | Switch to using ScopedRunnableMethodFactory::NewRunnableMethod(). Clean up some unnecessary AddRef()/Release() pairs. BUG=63692 TEST=existing Review URL: http://codereview.chromium.org/5559003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68206 0039d316-1c4b-4281-b951-d872f2087c98
* [ChromeOS] Make GaiaAutheFetcher parsing logic default to service_unavailablecmasone@chromium.org2010-12-023-2/+28
| | | | | | | | | | | | | | | | The logic that parses responses from Google Accounts servers was defaulting to INVALID_GAIA_CREDENTIALS, which indicate a successful communication with teh backend that resulted in a failing authentication attempt. Gibberish was thus interpreted this way. We should instead interpret gibberish as an unsuccessful communication attempt. BUG=chromium-os:8943 TEST=unit tests, installed on device and ran successful login and failed login autotests. Also tried live login. Review URL: http://codereview.chromium.org/5309007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68104 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed a bug with URLFetcher where retrying the request would append the new ↵sanjeevr@chromium.org2010-12-012-0/+54
| | | | | | | | | | | response data to the old one. BUG=None. TEST=Unit-tests Review URL: http://codereview.chromium.org/5322015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67815 0039d316-1c4b-4281-b951-d872f2087c98
* net: Remove typedef net::URLRequest URLRequest;tfarina@chromium.org2010-11-304-11/+12
| | | | | | | | | BUG=64263 TEST=compiled locally, trybots Review URL: http://codereview.chromium.org/5384002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67762 0039d316-1c4b-4281-b951-d872f2087c98
* Reland issue 5283001 (http://codereview.chromium.org/5283001) which was ↵sanjeevr@chromium.org2010-11-302-8/+18
| | | | | | | | | | | reverted because of Chrome OS test failures. BUG=None TEST=Unit tests. Review URL: http://codereview.chromium.org/5266004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67655 0039d316-1c4b-4281-b951-d872f2087c98
* net: Add namespace net to URLRequest and URLRequestJob classes.tfarina@chromium.org2010-11-281-5/+8
| | | | | | | | | BUG=64263 TEST=compiled locally and trybots Review URL: http://codereview.chromium.org/5298008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67497 0039d316-1c4b-4281-b951-d872f2087c98
* Implement exponential back-off mechanism.joi@chromium.org2010-11-255-394/+157
| | | | | | | | | | | | | | | | | | | | | | | Contributed by yzshen@google.com, original review http://codereview.chromium.org/4194001/ Implement exponential back-off mechanism. Enforce it at the URLRequestHttpJob level for all outgoing HTTP requests. The reason why to make this change is that we need back-off logic at a lower enough level to manage all outgoing HTTP traffic, so that the browser won't cause any DDoS attack. This change: 1) patches http://codereview.chromium.org/2487001/show, which is the exponential back-off implementation. 2) resolves conflicts with URLFetcher, by removing its own back-off logic: -- removes url_fetcher_protect.{h,cc}; -- integrates the sliding window mechanism of URLFetcherProtectEntry into RequestThrottlerEntry. 3) resolves conflicts with CloudPrintURLFetcher. 4) makes unit tests of CloudPrintURLFetcher, URLFetcher and URLRequest work. BUG=none TEST=pass all existing tests and also the newly-added request_throttler_unittest.cc Review URL: http://codereview.chromium.org/5276007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67375 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 67284 - Used a MessageLoopProxy to store the delegate's message loop ↵sanjeevr@chromium.org2010-11-242-17/+7
| | | | | | | | | | | | | | in URLFetcher::Core. This allows the thread that created the URLFetcher object to go away without causing the IO thread to crash. BUG=None TEST=None Review URL: http://codereview.chromium.org/5283001 TBR=sanjeevr@chromium.org Review URL: http://codereview.chromium.org/5382001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67320 0039d316-1c4b-4281-b951-d872f2087c98
* Used a MessageLoopProxy to store the delegate's message loop in ↵sanjeevr@chromium.org2010-11-242-7/+17
| | | | | | | | | | | URLFetcher::Core. This allows the thread that created the URLFetcher object to go away without causing the IO thread to crash. BUG=None TEST=None Review URL: http://codereview.chromium.org/5283001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67284 0039d316-1c4b-4281-b951-d872f2087c98
* Fix ThreadChecker to use Locks and not use scoped_ptr.willchan@chromium.org2010-11-211-0/+1
| | | | | | | | | | | | | | | It needs to synchronize its checks, since in order to assert correctly, it needs to make sure the thread id is synchronized on all threads. It doesn't need scoped_ptr. It was trying to use NULL to catch invalid thread ids. 0 is already assumed to be invalid (see base::Thread's use). Eliminating scoped_ptr fixes a valgrind/heapcheck issue where they don't follow LazyInstance objects' member pointers. So they think the ThreadChecker's member variable is leaked, even though the global object still has a pointer to it. Removing the scoped_ptr.h caused a bunch of other lame files to fail to compile. I had to fix those places. #include what you use please :( TBR=levin (I want to green the memory bots) BUG=none TEST=none Review URL: http://codereview.chromium.org/5180006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66915 0039d316-1c4b-4281-b951-d872f2087c98
* Allow sync with 2-factor StrongAuth accounts in ChromeOS.stevenjb@chromium.org2010-11-181-3/+6
| | | | | | | | | | | | | | 1. Detect login with 2-factor error and direct new accounts to chrome://settings/personal 2. Open BorwserSignin from 'Set Up Sync' button in sync settings 3. Enable constrained html dialog in BrowserSigning (was implemented for ChromeOS in a previous CL) 4. Attempt to log into sync when exiting screen locker if sync becomes disabled BUG=chromium-os:8752 TEST=see issue Review URL: http://codereview.chromium.org/4980005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66690 0039d316-1c4b-4281-b951-d872f2087c98
* Add a client-side phishing detection service class.noelutz@google.com2010-11-112-1/+144
| | | | | | | | | | | | This class is responsible for talking to the client-side detection servers and to fetch the machine learning model. BUG=none TEST=ClientSideDetectionServiceTest Review URL: http://codereview.chromium.org/3815014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65860 0039d316-1c4b-4281-b951-d872f2087c98
* openssl build fixes, and tidy the build filesjoth@chromium.org2010-11-111-3/+3
| | | | | | | | | | | Split out of http://codereview.chromium.org/4691003/ BUG=None TEST=builds... Review URL: http://codereview.chromium.org/4696004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65801 0039d316-1c4b-4281-b951-d872f2087c98
* Rename GaiaAuthenticator2 to GaiaAuthFetcher.tfarina@chromium.org2010-11-105-133/+132
| | | | | | | | | | | (Note: This was a TODO for chron). BUG=None TEST=trybots Review URL: http://codereview.chromium.org/4155010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65719 0039d316-1c4b-4281-b951-d872f2087c98
* Stop using DestructionObserver in OCSP code. Explicilty cancel all URLRequests.willchan@chromium.org2010-11-091-0/+14
| | | | | | | | | BUG=59630 TEST=none Review URL: http://codereview.chromium.org/4119020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65479 0039d316-1c4b-4281-b951-d872f2087c98
* Fix compile failure on GCCpkasting@chromium.org2010-11-081-1/+1
| | | | | | | | BUG=54274 TEST=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65426 0039d316-1c4b-4281-b951-d872f2087c98
* Make TestURLFetcherFactory able to remove fetchers from its map, and add an ↵pkasting@chromium.org2010-11-082-9/+24
| | | | | | | | | | ID accessor on TestURLFetcher, in preparation for more careful fetchre tracking in the GoogleURLTracker unittest. BUG=54274 TEST=none Review URL: http://codereview.chromium.org/4509002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65424 0039d316-1c4b-4281-b951-d872f2087c98
* In the webstore login api, we need to provide the gaia auth token to allow ↵johnnyg@chromium.org2010-11-013-2/+12
| | | | | | | | | | | token-based authentication, not the chromiumsync token. BUG=61490 TEST=webstore integration Review URL: http://codereview.chromium.org/4106016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64685 0039d316-1c4b-4281-b951-d872f2087c98
* chrome/common: Append base:: in the StringPrintf calls.tfarina@chromium.org2010-10-312-24/+24
| | | | | | | | | | | (Note: This is a TODO in string_util.h) BUG=None TEST=trybots Review URL: http://codereview.chromium.org/4111011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64567 0039d316-1c4b-4281-b951-d872f2087c98
* Followup on http://codereview.chromium.org/3815012/showbulach@chromium.org2010-10-292-2/+5
| | | | | | | | | | | Addresses a couple of nits (no functional changes). BUG=None TEST=None Review URL: http://codereview.chromium.org/4097005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64421 0039d316-1c4b-4281-b951-d872f2087c98
* Implement device token fetcherdanno@chromium.org2010-10-292-0/+3
| | | | | | | | | | | First step in mechanism for fetching cloud-based policy for CrOS. BUG=none TEST=DeviceTokenFetcher* Review URL: http://codereview.chromium.org/4121003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64388 0039d316-1c4b-4281-b951-d872f2087c98
* Initial support for built-in DNS resolver/cache.agayev@chromium.org2010-10-282-0/+68
| | | | | | | | | BUG=60149 TEST=None Review URL: http://codereview.chromium.org/4216001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64271 0039d316-1c4b-4281-b951-d872f2087c98
* Add support to test_server.py to restrict the SSL/TLS bulk encryption ↵rsleevi@chromium.org2010-10-281-2/+3
| | | | | | | | | | | algorithms via the command-line argument --ssl-alg. BUG=58831 TEST=Run test_server.py as an HTTPS server with --ssl-alg=rc4. Connect via openssl s_client -connect 127.0.0.1:1337 -cipher DEFAULT:\!RC4. Observe a connection failure. Connect with openssl s_client -connect 127.0.0.1:1337, observe that a ciphersuite that uses RC4 is negotiated. Review URL: http://codereview.chromium.org/3812007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64233 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 64202 - Initial support for built-in DNS resolver/cache.yusukes@chromium.org2010-10-282-68/+0
| | | | | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/3693001 TBR=agayev@chromium.org Review URL: http://codereview.chromium.org/4208002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64204 0039d316-1c4b-4281-b951-d872f2087c98
* Initial support for built-in DNS resolver/cache.agayev@chromium.org2010-10-282-0/+68
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/3693001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64202 0039d316-1c4b-4281-b951-d872f2087c98
* [CHrome OS] HOSTED accounts will not be allowed to log incmasone@chromium.org2010-10-252-17/+7
| | | | | | | | | | | | Instead of allowing HOSTED accounts to log in, but be flagged, we are instead detecting them and blocking them. Now, when a HOSTED account tries to log in, AuthenticateToLogin will call OnLoginFailure with USER_NOT_SIGNED_UP, indicating that the account in question is not allowed to use this device. BUG=chromium-os:7867 TEST=unit tests, install on device and attempt to log in with a HOSTED account. See that it fails. Review URL: http://codereview.chromium.org/3973010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63771 0039d316-1c4b-4281-b951-d872f2087c98
* More UI-related OpenSSL support.bulach@chromium.org2010-10-254-37/+145
| | | | | | | | | | | | | Refactors a few functions from nsNSSCertHelper into common. (no functional change when using NSS). Implements some of the x509_certificate_model functions for OpenSSL. BUG=none TEST=Compile w/ use_openssl=1, check that the fields in certificate viewer are displayed. Review URL: http://codereview.chromium.org/3815012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63723 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Revert "Const-ify RefCountedThreadSafe::AddRef and Release.""mpcomplete@chromium.org2010-10-223-7/+8
| | | | | | | | | chrome_frame failed to compile last time. I needed to add a "mutable" to a member variable in chrome_frame/metrics_service.cc. Review URL: http://codereview.chromium.org/3971004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63574 0039d316-1c4b-4281-b951-d872f2087c98
* Make sure the service process kills all URLFetcher::Cores on shutdown.willchan@chromium.org2010-10-221-14/+0
| | | | | | | | | | | | | I'm not sure this is strictly necessary, since it may be that due to how the service process uses URLFetcher, that all fetches are already canceled by the time its IO thread shuts down. But this makes absolutely sure this is the case. Good practice in any case. This also makes it completely safe for us to have URLFetcher::Core stop using the DestructionObserver. BUG=59630 TEST=existing Review URL: http://codereview.chromium.org/3969006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63568 0039d316-1c4b-4281-b951-d872f2087c98
* [Chrome OS] On first login, determine if an account is a HOSTED accountcmasone@chromium.org2010-10-221-3/+17
| | | | | | | | | | | Adds |is_hosted| to GaiaAuthConsumer::ClientLoginResult, so the recipient of a CLR struct can tell if the authenticated account is hosted or not. BUG=chromium-os:7867 TEST=unit tests, install on device and log in with hosted account. Look for log message indicating HOSTED login Review URL: http://codereview.chromium.org/3898003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63524 0039d316-1c4b-4281-b951-d872f2087c98
* Kill all URLFetcher URLRequests on shutdown.willchan@chromium.org2010-10-222-14/+99
| | | | | | | | | BUG=59630 TEST=none Review URL: http://codereview.chromium.org/3826016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63463 0039d316-1c4b-4281-b951-d872f2087c98
* Convert LOG(INFO) to VLOG(1) - chrome/common/.pkasting@chromium.org2010-10-221-7/+8
| | | | | | | | | | | | This also removes LOG_RESOURCE_REQUESTS and all associated code. Also remove some "using"s, fix non-const ref (style violation), remove some "else" after "return", and remove some extra {}s. BUG=none TEST=none Review URL: http://codereview.chromium.org/3941001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63462 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Const-ify RefCountedThreadSafe::AddRef and Release."levin@chromium.org2010-10-223-8/+7
| | | | | | This reverts commit b7ce919957536ceb0cfac1709bc779fd086b6ce8. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63459 0039d316-1c4b-4281-b951-d872f2087c98
* Const-ify RefCountedThreadSafe::AddRef and Release.mpcomplete@chromium.org2010-10-223-7/+8
| | | | | | Review URL: http://codereview.chromium.org/3869003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63457 0039d316-1c4b-4281-b951-d872f2087c98
* In a fake SSLInfo, set security_bits to -1 instead of 0 wtc@chromium.org2010-10-151-1/+1
| | | | | | | | | | | | because the encryption strength is unknown (0 means no encryption). R=finnur BUG=53366,13049 TEST=no unit test failures Review URL: http://codereview.chromium.org/3749003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62785 0039d316-1c4b-4281-b951-d872f2087c98
* Reland fix for OCSP startup race.willchan@chromium.org2010-10-122-3/+15
| | | | | | | | | | | | | This is a revert of r62107 which is a revert of r62105 which is a revert of r60753 which is a revert of r60739 which is a revert of r60025 which is a revert of r59972 which is a revert of r59570 which is a revert of r59511 which is a revert of r59299 which is a revert of r59289. The most recent reverts have all been due to ChromiumOS leaks which I believe to be fixed after fixing bug 58572, since I tested the combined fix in r62105 which was green on all ChromiumOS valgrind bots. BUG=55940 TEST=Run a debug build on Linux. 10% or so of startups used to crash on startup. Shouldn't happen anymore. Review URL: http://codereview.chromium.org/3660010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62256 0039d316-1c4b-4281-b951-d872f2087c98
* Rename ChromeThread to BrowserThread Part16:tfarina@chromium.org2010-10-111-1/+1
| | | | | | | | | | | - Rename entries starting from shell_integration.cc. More 30 files covered. BUG=56926 TEST=trybots Review URL: http://codereview.chromium.org/3708001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62136 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 62105 - Test change to fix a leak. Will revert after getting in a ↵willchan@chromium.org2010-10-102-15/+3
| | | | | | | | | | | | | run on ChromiumOS valgrind bots. Revert "Revert 60739 (still leaks on ChromiumOS!) - Reland r59972: Eagerly set the IO loop used for OCSP." Review URL: http://codereview.chromium.org/3681003 TBR=willchan@chromium.org Review URL: http://codereview.chromium.org/3669003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62107 0039d316-1c4b-4281-b951-d872f2087c98
* Test change to fix a leak. Will revert after getting in a run on ChromiumOS ↵willchan@chromium.org2010-10-102-3/+15
| | | | | | | | | | valgrind bots. Revert "Revert 60739 (still leaks on ChromiumOS!) - Reland r59972: Eagerly set the IO loop used for OCSP." Review URL: http://codereview.chromium.org/3681003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62105 0039d316-1c4b-4281-b951-d872f2087c98
* Decouples certificates viewers from NSS to prepare support for OpenSSL.bulach@chromium.org2010-10-085-0/+769
| | | | | | | | | | | | This change is a pre-requisite for http://codereview.chromium.org/3529008/show There are no functional changes, it's only refactoring existing code. BUG=None TEST=Go to an https:// page and check the certificate info. Review URL: http://codereview.chromium.org/3565006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61944 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Initialize data members in constructor.jhawkins@chromium.org2010-10-051-1/+2
| | | | | | | | | | | | CID=12798, 12835, 12921, 12922, 12923, 13083, 1408, 7026, 7027, 13052, 13058, 8295, 2289, 13026, 13133, 13233, 13184, 13183 BUG=none TEST=none Patch by Kausalya Madhusudhanan <kmadhusu@chromium.org> Review URL: http://codereview.chromium.org/3567011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61448 0039d316-1c4b-4281-b951-d872f2087c98
* Sync shouldn't allow hosted accounts, but other clients of ↵johnnyg@chromium.org2010-10-023-23/+47
| | | | | | | | | | | GaiaAuthenticator2 still should. BUG=chromiumos:7296 TEST=see bug Review URL: http://codereview.chromium.org/3599010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61318 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Move ctors/dtors into implementation files. Adds ctors/dtors to ↵erg@google.com2010-09-306-5/+27
| | | | | | | | | | | | | | non-POD structs. Cuts ~2MB off our .a files (Debug, Linux). Also added the "virtual" keyword on a whole bunch of virtual dtors that were missing it. BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3522004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61100 0039d316-1c4b-4281-b951-d872f2087c98
* Fix problems with incorrect passwords and error states due to HTML and .h ↵johnnyg@chromium.org2010-09-281-14/+18
| | | | | | | | | | | file mismatch on error codes. BUG=53151 TEST=invalid password and captcha state in sync login Review URL: http://codereview.chromium.org/3429024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60801 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 60739 (still leaks on ChromiumOS!) - Reland r59972: Eagerly set the ↵willchan@chromium.org2010-09-281-17/+1
| | | | | | | | | | | | | | | | | | IO loop used for OCSP. ChromeOS will create a special Profile for login. Previously, OCSP initialization was done for the "default" ChromeURLRequestContext for each Profile. Since we can have multiple profiles, this causes the initialization (and uninitialization) to happen multiple times, which causes problems for OCSP since we use statics. The solution is to identify the "main" Profile. We create said Profile in BrowserMain. I add an "is_main_" variable to URLRequestContextGetter and URLRequestContext, so that only the "main" URLRequestContext will initialize OCSP. The only change from r59972 is the location that I set_is_main(). I set it when we set the default request context. This is because, by the time I had called set_is_main() in BrowserMain, the ChromeURLRequestContextGetter was already created (it gets created during the CreateProfile() function). So I was too late. BUG=55940 TEST=Manual Review URL: http://codereview.chromium.org/3391028 TBR=willchan@chromium.org Review URL: http://codereview.chromium.org/3421039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60753 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r59972: Eagerly set the IO loop used for OCSP.willchan@chromium.org2010-09-281-1/+17
| | | | | | | | | | | | | ChromeOS will create a special Profile for login. Previously, OCSP initialization was done for the "default" ChromeURLRequestContext for each Profile. Since we can have multiple profiles, this causes the initialization (and uninitialization) to happen multiple times, which causes problems for OCSP since we use statics. The solution is to identify the "main" Profile. We create said Profile in BrowserMain. I add an "is_main_" variable to URLRequestContextGetter and URLRequestContext, so that only the "main" URLRequestContext will initialize OCSP. The only change from r59972 is the location that I set_is_main(). I set it when we set the default request context. This is because, by the time I had called set_is_main() in BrowserMain, the ChromeURLRequestContextGetter was already created (it gets created during the CreateProfile() function). So I was too late. BUG=55940 TEST=Manual Review URL: http://codereview.chromium.org/3391028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60739 0039d316-1c4b-4281-b951-d872f2087c98
* Fix typo in a function name.pkasting@chromium.org2010-09-273-5/+5
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3467017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60727 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-06 15:46:18 +0000