summaryrefslogtreecommitdiffstats
path: root/net/http/http_transaction_unittest.h
Commit message (Collapse)AuthorAgeFilesLines
* Rename http_transaction_unittest to http_transaction_test_util.eustas@chromium.org2014-05-141-306/+0
| | | | | | | | | | | | And move it to test support target. Actually this file contains only utilities for tests, but not tests themselves. BUG=245436 Review URL: https://codereview.chromium.org/280763003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270354 0039d316-1c4b-4281-b951-d872f2087c98
* net: Avoid recursion in URLRequestJob::ReadFilteredData.rvargas@chromium.org2014-05-031-1/+2
| | | | | | | | | | | | | | | | There's no need to use recursion when getting the data from a filter. This version uses a loop instead. The issue with recursion is that it opens the door to stack exhaustion if there is no way to limit its depth. In this case, that would be a hard thing to do given that it basically depends on interactions outside of our control. BUG=366149 Review URL: https://codereview.chromium.org/267793002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@268030 0039d316-1c4b-4281-b951-d872f2087c98
* Don't cache non-redirect HTTP responses when a redirect is forcedmmenke@chromium.org2014-03-241-0/+3
| | | | | | | | | | | | | | by a NetworkDelegate in OnHeadersReceived. This regressed in https://chromiumcodereview.appspot.com/23710059. Fortunately, no NetworkDelegate currently does this, but that is going to change. BUG=354508 Review URL: https://codereview.chromium.org/208683002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@258983 0039d316-1c4b-4281-b951-d872f2087c98
* Minor cleanup of SetQuicServerInfo.rtenneti@chromium.org2014-01-251-0/+3
| | | | | | | | R=wtc@chromium.org, rvargas@chromium.org Review URL: https://codereview.chromium.org/143073006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247095 0039d316-1c4b-4281-b951-d872f2087c98
* Allows deferral of a URLRequest just before talking to the network, atjkarlin@chromium.org2014-01-081-0/+5
| | | | | | | | | | | | | DoCreateStream time in the HttpNetworkTransaction. This hook allows for experimentation with the ResourceScheduler so that it can defer after first checking the cache, gathering the cookies, and starting the WebRequest but before going to the network. BUG=328741 Review URL: https://codereview.chromium.org/122453002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@243568 0039d316-1c4b-4281-b951-d872f2087c98
* Http Cache: Attempt to authenticate requests that originate in the cache.rvargas@chromium.org2014-01-041-0/+4
| | | | | | | | | | | | | | | | | | | The cache can decompose some requests and issue a variable number of network request for a single user-generated request. When network authentication is part of the picture, we may receive authentication failures after completing Start() processing for the "main" request, so doing regular auth is not an option (going to the caller to select credentials etc). This CL attempts automatic authentication for the extra requests, and improves error handling so that when everything else fails the partially cached content is removed from the cache. BUG=316602 R=wtc@chromium.org Review URL: https://codereview.chromium.org/102113006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242972 0039d316-1c4b-4281-b951-d872f2087c98
* Remove HttpTransactionDelegate and all users (LoadTimeStats).rvargas@chromium.org2013-12-271-2/+1
| | | | | | | | | | | | | These metrics are not used anymore. TEST=builds BUG= TBR=mkosiba@chromium.org (webview) Review URL: https://codereview.chromium.org/121453002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242643 0039d316-1c4b-4281-b951-d872f2087c98
* Implement GetTotalReceivedBytes for http transaction.eustas@chromium.org2013-12-241-0/+3
| | | | | | | | BUG=111052 Review URL: https://codereview.chromium.org/99723004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242433 0039d316-1c4b-4281-b951-d872f2087c98
* Add a SetWebSocketHandshakeStreamFactory() method to the HttpTransaction ↵ricea@chromium.org2013-11-141-0/+8
| | | | | | | | | | | | | | | base class so that URLRequestHttpJob can pass through the WebSocketHandshakeStreamBase::Factory object. Implement OnWebSocketHandshakeStreamReady(). BUG=315027 TEST=net_unittests Review URL: https://codereview.chromium.org/23856018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@235016 0039d316-1c4b-4281-b951-d872f2087c98
* Add GetFullRequestHeaders, from URLRequestJob to HttpNetworkTransaction.ttuttle@chromium.org2013-06-201-0/+4
| | | | | | | | | | | | | | | | | Dev Tools displays the raw request and response headers. Right now, it gets them by snarfing them out of the NetLog, which is bad, as the NetLog is for huamn consumption only. I'm trying to refactor the DevToolsNetLogObserver away; providing an alternate, supported path to get the request headers is the first step. BUG=196304 TEST=added to URLRequest, HttpTransaction, and HttpNetworkTransaction unittests Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=199535 Review URL: https://chromiumcodereview.appspot.com/12621011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@207341 0039d316-1c4b-4281-b951-d872f2087c98
* Use a direct include of strings headers in net/http/.avi@chromium.org2013-06-111-1/+1
| | | | | | | | | | BUG=247723 TEST=none TBR=ben@chromium.org Review URL: https://chromiumcodereview.appspot.com/15906017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205594 0039d316-1c4b-4281-b951-d872f2087c98
* net: Return LoadTiming information in the case of a cache hit.mmenke@chromium.org2013-05-151-0/+6
| | | | | | | | | | | | | If the request went over the wire before headers were returned, return information about the last over the wire request. Otherwise, set send times to be when the cache entry was opened, and receive_header_start to be when the headers were read. BUG=239842 Review URL: https://chromiumcodereview.appspot.com/14625012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@200356 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 199535 "Add GetFullRequestHeaders, from URLRequestJob to ..."ttuttle@chromium.org2013-05-101-4/+0
| | | | | | | | | | | | | | | | | | | | | > Add GetFullRequestHeaders, from URLRequestJob to HttpNetworkTransaction. > > Dev Tools displays the raw request and response headers. Right now, it gets > them by snarfing them out of the NetLog, which is bad, as the NetLog is for > huamn consumption only. I'm trying to refactor the DevToolsNetLogObserver > away; providing an alternate, supported path to get the request headers is the > first step. > > BUG=196304 > TEST=added to URLRequest, HttpTransaction, and HttpNetworkTransaction unittests > > Review URL: https://chromiumcodereview.appspot.com/12621011 TBR=ttuttle@chromium.org Review URL: https://codereview.chromium.org/15054003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@199553 0039d316-1c4b-4281-b951-d872f2087c98
* Add GetFullRequestHeaders, from URLRequestJob to HttpNetworkTransaction.ttuttle@chromium.org2013-05-101-0/+4
| | | | | | | | | | | | | | | Dev Tools displays the raw request and response headers. Right now, it gets them by snarfing them out of the NetLog, which is bad, as the NetLog is for huamn consumption only. I'm trying to refactor the DevToolsNetLogObserver away; providing an alternate, supported path to get the request headers is the first step. BUG=196304 TEST=added to URLRequest, HttpTransaction, and HttpNetworkTransaction unittests Review URL: https://chromiumcodereview.appspot.com/12621011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@199535 0039d316-1c4b-4281-b951-d872f2087c98
* Cache failover to LOAD_PREFERRING_CACHE if network response suggests offline.rdsmith@chromium.org2013-03-231-0/+3
| | | | | | | | | | | | (Controlled by new load flag.) BUG=2204 R=rvargas@chromium.org Review URL: https://chromiumcodereview.appspot.com/12310075 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@190020 0039d316-1c4b-4281-b951-d872f2087c98
* [Net] Propagate priority changes from URLRequest to HttpTransactionakalin@chromium.org2013-03-221-1/+32
| | | | | | | | | | | | | | | This is in preparation for propagating priority changes from ResourceScheduler all the way to HostResolver and ClientSocketPool. Add some NetLog events and parameters for priority. BUG=166689 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=189829 Review URL: https://codereview.chromium.org/12701011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189894 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 189829 "[Net] Propagate priority changes from URLRequest ..."msw@chromium.org2013-03-221-32/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See heapcheck failures: http://build.chromium.org/p/chromium.memory.fyi/builders/Linux%20Heapcheck/builds/25563 http://build.chromium.org/p/chromium.memory.fyi/builders/Linux%20Heapcheck/builds/25563/steps/heapcheck%20test%3A%20net/logs/stdio Example (one of many): Leak of 792 bytes in 1 objects allocated from: @ 10d675b URLRequestFtpJobPriorityTest @ 10d68e7 URLRequestFtpJobPriorityTest_SetPriorityBasic_Test @ 10de383 CreateTest Suppression (error hash=#0A68FF60C5D043B8#): { <insert_a_suppression_name_here> Heapcheck:Leak fun:URLRequestFtpJobPriorityTest fun:URLRequestFtpJobPriorityTest_SetPriorityBasic_Test fun:CreateTest } > [Net] Propagate priority changes from URLRequest to HttpTransaction > > This is in preparation for propagating priority changes from > ResourceScheduler all the way to HostResolver and ClientSocketPool. > > Add some NetLog events and parameters for priority. > > BUG=166689 > TBR=simonjam@chromium.org,sky@chromium.org > > Review URL: https://codereview.chromium.org/12701011 TBR=akalin@chromium.org Review URL: https://codereview.chromium.org/12676022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189872 0039d316-1c4b-4281-b951-d872f2087c98
* [Net] Propagate priority changes from URLRequest to HttpTransactionakalin@chromium.org2013-03-221-1/+32
| | | | | | | | | | | | | | This is in preparation for propagating priority changes from ResourceScheduler all the way to HostResolver and ClientSocketPool. Add some NetLog events and parameters for priority. BUG=166689 TBR=simonjam@chromium.org,sky@chromium.org Review URL: https://codereview.chromium.org/12701011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189829 0039d316-1c4b-4281-b951-d872f2087c98
* [Net] Separate out priority field from HttpRequestInfoakalin@chromium.org2013-03-191-2/+6
| | | | | | | | | | | | | | This is in preparation for supporting reprioritization; HttpRequestInfo is assumed to be an unchanging struct, so priority can't live in it if it is something that can change. Also add DEFAULT_PRIORITY value and replace some uses of LOWEST with it. BUG=166689 Review URL: https://codereview.chromium.org/12833008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189099 0039d316-1c4b-4281-b951-d872f2087c98
* LoadTiming in net, part 3.mmenke@chromium.org2013-01-211-0/+3
| | | | | | | | | | | | | | | Add send and receive times to HttpNetworkTransaction. Add LoadTimingInfo accessors to Http*Transaction, URLRequestJob, and URLRequest classes. Proxy resolution times still need to be added. BUG=77446 Review URL: https://chromiumcodereview.appspot.com/11929017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@177942 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 153131 (Histograms showed it doesn't help much)mmenke@chromium.org2012-09-111-5/+0
| | | | | | | | | | | | | | | | | | Reland of http://codereview.chromium.org/10854204/, which I reverted because of a bug in its unit tests. Retry failed network requests. R=willchan@chomium.org BUG=143425 Review URL: https://chromiumcodereview.appspot.com/10872044 TBR=mmenke@chromium.org Review URL: https://chromiumcodereview.appspot.com/10933022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156041 0039d316-1c4b-4281-b951-d872f2087c98
* net: Return size of upload as well as position from ↵hashimoto@chromium.org2012-08-301-1/+1
| | | | | | | | | | | | | | HttpTransaction::GetUploadProgress() Add net::UploadProgress Change return type of GetUploadSize() from uint64 to UploadProgress. BUG=112607 TEST=net_unittests Review URL: https://chromiumcodereview.appspot.com/10834178 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154063 0039d316-1c4b-4281-b951-d872f2087c98
* Reland of http://codereview.chromium.org/10854204/, whichmmenke@chromium.org2012-08-241-0/+5
| | | | | | | | | | | | | I reverted because of a bug in its unit tests. Retry failed network requests. R=willchan@chomium.org BUG=143425 Review URL: https://chromiumcodereview.appspot.com/10872044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153131 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 153025 - Automatically retry failed network requests.mmenke@chromium.org2012-08-231-5/+0
| | | | | | | | | | | | | | This is a temporary experiment to see if this behavior is worth implementing in a cleaner fashion. BUG=143425 Review URL: https://chromiumcodereview.appspot.com/10854204 TBR=mmenke@chromium.org Review URL: https://chromiumcodereview.appspot.com/10883011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153031 0039d316-1c4b-4281-b951-d872f2087c98
* Automatically retry failed network requests.mmenke@chromium.org2012-08-231-0/+5
| | | | | | | | | | | This is a temporary experiment to see if this behavior is worth implementing in a cleaner fashion. BUG=143425 Review URL: https://chromiumcodereview.appspot.com/10854204 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153025 0039d316-1c4b-4281-b951-d872f2087c98
* Adding histograms showing fraction of page load times spent in the disk ↵tburkard@chromium.org2012-07-301-2/+3
| | | | | | | | | | cache, for full and partial page loads. R=mmenke@chromium.org, jam@chromium.org, willchan@chromium.org, brettw@chromium.org BUG=139210 Review URL: https://chromiumcodereview.appspot.com/10736066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149007 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the rest of #pragma once in one big CL.ajwong@chromium.org2012-07-111-1/+0
| | | | | | | | | For context see this thread: https://groups.google.com/a/chromium.org/forum/?fromgroups#!topic/chromium-dev/RMcVNGjB4II TBR=thakis,pkasting,jam git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146163 0039d316-1c4b-4281-b951-d872f2087c98
* Migrate HttpCache adn HttpCacheTransaction to base::Bind().ajwong@chromium.org2011-12-231-3/+2
| | | | | | | | | BUG=none TEST=existing Review URL: http://codereview.chromium.org/9025033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115675 0039d316-1c4b-4281-b951-d872f2087c98
* base::Bind: Convert most of net/http.jhawkins@chromium.org2011-12-201-8/+8
| | | | | | | | | | BUG=none TEST=none R=csilv Review URL: http://codereview.chromium.org/8990001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115220 0039d316-1c4b-4281-b951-d872f2087c98
* Add OVERRIDE to net/.avi@chromium.org2011-11-161-4/+5
| | | | | | | | BUG=104314 Review URL: http://codereview.chromium.org/8568021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110231 0039d316-1c4b-4281-b951-d872f2087c98
* Use AuthCredentials throughout the network stack instead of username/password.cbentzel@chromium.org2011-10-281-14/+16
| | | | | | | | This is a refactor only - no behavior change should happen. Review URL: http://codereview.chromium.org/8340026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107766 0039d316-1c4b-4281-b951-d872f2087c98
* base::Bind conversion for net/http/...adamk@chromium.org2011-10-201-1/+1
| | | | | | | | | R=willchan@chromium.org Review URL: http://codereview.chromium.org/8344066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106428 0039d316-1c4b-4281-b951-d872f2087c98
* Begin CompletionCallback switchover.willchan@chromium.org2011-10-011-7/+7
| | | | | | | | | | | | | Rename CompletionCallback to OldCompletionCallback in preparation for introducing a new CompletionCallback based on base::Callback. Also renames other CompletionCallback types like CancelableCompletionCallback and TestCompletionCallback and CompletionCallbackImpl. All using sed with s/CompletionCallback/OldCompletionCallback/g. BUG=98719 TEST=none Review URL: http://codereview.chromium.org/8070013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103650 0039d316-1c4b-4281-b951-d872f2087c98
* For the SSL cert status, convert anonymous enum that gives bit values into a ↵pkasting@chromium.org2011-09-231-1/+1
| | | | | | | | | | | | | | typedefed uint32. This allows code all over Chromium to use an explicit type instead of "int". This also means the individual named bit constants themselves have the same explicit type. I find the resulting code to be noticeably clearer. This also exposed a bug in SSLErrorInfo::GetErrorsForCertStatus() where not having an explicit type allowed a function argument ordering bug to creep in, so I claim this is safer too. Normally this makes things like DCHECK_EQ() unhappy, but when I'd originally tested this I didn't seem to need to make any changes due to that. Will be watching the trybots... The original motiviation for this change was to find a way to eliminate some cases of passing anonymous-typed values as template arguments (which happens when you use a value from the enum in e.g. EXPECT_EQ()), which is technically illegal in C++03, though we don't warn about it. Simply naming the enum would have done this, but this would have encouraged readers to actually use the enum name as a type, which for a bitfield is inappropriate for the reason given in the first paragraph. BUG=92247 TEST=Compiles Review URL: http://codereview.chromium.org/7969023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102415 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 102322 - For the SSL cert status, convert anonymous enum that gives ↵pkasting@chromium.org2011-09-221-1/+1
| | | | | | | | | | | | | | | | | | | bit values into a typedefed uint32. This allows code all over Chromium to use an explicit type instead of "int". (This isn't possible by simply naming the enum as technically the enum doesn't define all of the possible combinations of bits.) This also means the individual named bit constants themselves have the same explicit type. I find the resulting code to be noticeably clearer. This also exposed a bug in SSLErrorInfo::GetErrorsForCertStatus() where not having an explicit type allowed a function argument ordering bug to creep in, so I claim this is safer too. I also added CERT_STATUS_NO_ERROR in place of "0" as a magic number. Normally this makes things like DCHECK_EQ() unhappy, but when I'd originally tested this I didn't seem to need to make any changes due to that. Will be watching the trybots... The original motiviation for this change was to find a way to eliminate some cases of passing anonymous-typed values as template arguments (which happens when you use a value from the enum in e.g. EXPECT_EQ()), which is technically illegal in C++03, though we don't warn about it. Simply naming the enum would have done this, but this would have encouraged readers to actually use the enum name as a type, which for a bitfield is inappropriate for the reason given in the first paragraph. BUG=92247 TEST=Compiles Review URL: http://codereview.chromium.org/7819009 TBR=pkasting@chromium.org Review URL: http://codereview.chromium.org/7995014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102325 0039d316-1c4b-4281-b951-d872f2087c98
* For the SSL cert status, convert anonymous enum that gives bit values into a ↵pkasting@chromium.org2011-09-221-1/+1
| | | | | | | | | | | | | | | | typedefed uint32. This allows code all over Chromium to use an explicit type instead of "int". (This isn't possible by simply naming the enum as technically the enum doesn't define all of the possible combinations of bits.) This also means the individual named bit constants themselves have the same explicit type. I find the resulting code to be noticeably clearer. This also exposed a bug in SSLErrorInfo::GetErrorsForCertStatus() where not having an explicit type allowed a function argument ordering bug to creep in, so I claim this is safer too. I also added CERT_STATUS_NO_ERROR in place of "0" as a magic number. Normally this makes things like DCHECK_EQ() unhappy, but when I'd originally tested this I didn't seem to need to make any changes due to that. Will be watching the trybots... The original motiviation for this change was to find a way to eliminate some cases of passing anonymous-typed values as template arguments (which happens when you use a value from the enum in e.g. EXPECT_EQ()), which is technically illegal in C++03, though we don't warn about it. Simply naming the enum would have done this, but this would have encouraged readers to actually use the enum name as a type, which for a bitfield is inappropriate for the reason given in the first paragraph. BUG=92247 TEST=Compiles Review URL: http://codereview.chromium.org/7819009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102322 0039d316-1c4b-4281-b951-d872f2087c98
* net: Notify the http job and cache transaction about a filterrvargas@google.com2011-08-121-2/+11
| | | | | | | | | | completing the request (returning 0 bytes from a read). BUG=91898 TEST=net_unittests Review URL: http://codereview.chromium.org/7569027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96576 0039d316-1c4b-4281-b951-d872f2087c98
* Kill URLRequestJobTracker.willchan@chromium.org2011-05-221-2/+1
| | | | | | | | | | BUG=81160 TEST=none Review URL: http://codereview.chromium.org/7043007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@86241 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up net unit testing code.erg@google.com2011-01-261-177/+40
| | | | | | | | | | | | | | | | | | | | (Reapply r72562 with willchan's nits + locally tested shlib fixes + removing duplicate code that was partially responsible for the failure.) - Move code included from blah_unittest.h (where blah_unittest.cc has actual unittests) into their own files, often completely out-of-lining the definitions. - Remove anonymous namespaces from headers. - Reorder method declarations. - Make other unit test link against net_test_support instead of reaching in and including .h files in their targets directly. BUG=68682 TEST=compiles First Review URL: http://codereview.chromium.org/6264013 Review URL: http://codereview.chromium.org/6248021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72682 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Clean up net unit testing code." since it breaks the shlib builder.erg@google.com2011-01-251-40/+177
| | | | | | | | | | This reverts commit 006f377bbb65ece3490b7c76e58e0dc4cb330909 (r72562). BUG=68682 TEST=compiles TBR=thomasvl git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72564 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up net unit testing code.erg@google.com2011-01-251-177/+40
| | | | | | | | | | | | | | | | | - Move code included from blah_unittest.h (where blah_unittest.cc has actual unittests) into their own files, often completely out-of-lining the definitions. - Remove anonymous namespaces from headers. - Reorder method declarations. - Make other unit test link against net_test_support instead of reaching in and including .h files in their targets directly. BUG=68682 TEST=compiles Review URL: http://codereview.chromium.org/6264013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72562 0039d316-1c4b-4281-b951-d872f2087c98
* net: Append base:: in the StringPrintf calls.tfarina@chromium.org2010-09-251-3/+4
| | | | | | | | | | | (Note: This is a TODO in string_util.h) BUG=None TEST=trybots Review URL: http://codereview.chromium.org/3390026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60555 0039d316-1c4b-4281-b951-d872f2087c98
* Net: Convert username and password to string16.thestig@chromium.org2010-07-291-3/+4
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3040016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54101 0039d316-1c4b-4281-b951-d872f2087c98
* `#pragma once` for app, base, chrome, gfx, ipc, net, skia, viewsthakis@chromium.org2010-07-261-0/+1
| | | | | | | | | BUG=50273 TEST=everything still builds, build is 10% faster on windows, same speed on mac/linux TBR: erg git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53716 0039d316-1c4b-4281-b951-d872f2087c98
* Http Cache: chain GetLoadState from transactions waiting forrvargas@google.com2010-07-151-1/+2
| | | | | | | | | | | | | | | | | | access to an entry to the transaction that has exclusive access to that entry. This means that requests that are blocked on the single writer lock of the http cache will report the load state of the request that has the lock, and WAITING_FOR_CACHE is reserved for requests that are actually waiting for the entry to be created by the cache. BUG=43582 TEST=net_unittests Review URL: http://codereview.chromium.org/2892010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52557 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 52334 - Http Cache: chain GetLoadState from transactions waiting forrvargas@google.com2010-07-141-2/+1
| | | | | | | | | | | | | | | | | | | | | | access to an entry to the transaction that has exclusive access to that entry. This means that requests that are blocked on the single writer lock of the http cache will report the load state of the request that has the lock, and WAITING_FOR_CACHE is reserved for requests that are actually waiting for the entry to be created by the cache. BUG=43582 TEST=net_unittests Review URL: http://codereview.chromium.org/2976004 TBR=rvargas@google.com Review URL: http://codereview.chromium.org/2921010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52405 0039d316-1c4b-4281-b951-d872f2087c98
* Http Cache: chain GetLoadState from transactions waiting forrvargas@google.com2010-07-141-1/+2
| | | | | | | | | | | | | | | | | | | access to an entry to the transaction that has exclusive access to that entry. This means that requests that are blocked on the single writer lock of the http cache will report the load state of the request that has the lock, and WAITING_FOR_CACHE is reserved for requests that are actually waiting for the entry to be created by the cache. BUG=43582 TEST=net_unittests Review URL: http://codereview.chromium.org/2976004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52334 0039d316-1c4b-4281-b951-d872f2087c98
* Auto-format style pass over files.cbentzel@chromium.org2010-05-041-3/+3
| | | | | | | | | | | This is a refactor (actually reformat) only - no behavior change in place. BUG=NONE TEST=net_unittests.exe Review URL: http://codereview.chromium.org/1800003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46349 0039d316-1c4b-4281-b951-d872f2087c98
* Use HttpRequestHeaders for extra_headers.willchan@chromium.org2010-04-201-1/+1
| | | | | | | | BUG=22588 Review URL: http://codereview.chromium.org/1604011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45096 0039d316-1c4b-4281-b951-d872f2087c98
* Generalize the net module's LoadLog facility from a passive container, to an ↵eroman@chromium.org2010-03-161-3/+4
| | | | | | | | | | | | | | | | event stream (NetLog). This makes it possible to associate a single NetLog with a URLRequestContext, and then attach observers to that log to watch the stream of events. This changelist attempts to do the most direct translation, so there will be subsequent iterations to clean up. The user-visible behavior should remain unchanged. BUG=37421 Review URL: http://codereview.chromium.org/848006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41689 0039d316-1c4b-4281-b951-d872f2087c98