summaryrefslogtreecommitdiffstats
path: root/net/url_request
Commit message (Collapse)AuthorAgeFilesLines
* Add net/base/test_data_directory.htoyoshim@chromium.org2012-11-011-0/+1
| | | | | | | | | | | | - move GetTestCertsDirectory() from net/base/certs_test_util BUG=157531 TEST=build all TBR=wtc@chromium.org Review URL: https://chromiumcodereview.appspot.com/11342034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165348 0039d316-1c4b-4281-b951-d872f2087c98
* Account for server vs host clock skew in cookie expiration times.pauljensen@chromium.org2012-11-013-0/+103
| | | | | | | | | | | | | When setting a cookie's expiration time in the cookie store we need to take into account any difference between the HTTP server and the host machine's real time clock. This reverts 159685 which reverted 146616. BUG=135131 TEST=net_unittests --gtest_filter=CookieMonster/CookieStoreTest/0.TestCookieDeletion Review URL: https://chromiumcodereview.appspot.com/11339032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165323 0039d316-1c4b-4281-b951-d872f2087c98
* Switch the TCP reads on Windows to use non-blocking/non-async I/O.pmeenan@google.com2012-10-311-0/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Overlapped I/O was introducing delays when the networking stack did not have enough data to fill the receive buffer. This can be seen when loading pssplayground.com/ksimbili/webp.html using a DSL connection profile on WebPagetest: http://www.webpagetest.org/result/120830_MS_414849a6aa055bb853e7e5d51e1b29d8/ and manifests and increasingly long Time to First Byte for requests further down the waterfall (expected values are < 90ms and it was going over 150ms). It is configured as a 50% field trial and can be forced through the command-line for testing: --overlapped-reads=on - default/existing behavior --overlapped-reads=off - new read implementation Trial-specific histograms are reported for page load times and http request times. Specifically: PLT.Abandoned PLT.LoadType PLT.BeginToFinish_NormalLoad PLT.BeginToFinish_LinkLoadNormal PLT.BeginToFinish_LinkLoadReload PLT.BeginToFinish_LinkLoadStaleOk Net.HttpJob.TotalTime Net.HttpJob.TotalTimeSuccess Net.HttpJob.TotalTimeCancel Net.HttpJob.TotalTimeCached Net.HttpJob.TotalTimeNotCached Review URL: https://chromiumcodereview.appspot.com/10916016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165170 0039d316-1c4b-4281-b951-d872f2087c98
* Process only the first Strict-Transport-Security header.palmer@chromium.org2012-10-303-0/+68
| | | | | | | | | | In accordance with the specification. BUG=156147 Review URL: https://chromiumcodereview.appspot.com/11192045 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165013 0039d316-1c4b-4281-b951-d872f2087c98
* Fix trivial bug in URLFetcherCore::SetStopOnRedirect() that ignored the ↵pkasting@chromium.org2012-10-291-1/+1
| | | | | | | | | | supplied argument. BUG=none TEST=none Review URL: https://codereview.chromium.org/11343023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@164737 0039d316-1c4b-4281-b951-d872f2087c98
* Add upload identifier to URLRequest::Start net log event.cbentzel@chromium.org2012-10-273-1/+7
| | | | | | | | | BUG=145070 Review URL: https://chromiumcodereview.appspot.com/11276050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@164550 0039d316-1c4b-4281-b951-d872f2087c98
* Improve URLRequestContextBuilder for use in android_webviewboliu@chromium.org2012-10-273-16/+31
| | | | | | | | | | | | | | Implemented [Enable|Disable]HttpCache methods. Added OS_ANDROID for set_proxy_config_service guard. Added set_network_delegate method. Also fix a double owning issue introduced by me in r164387. BUG= Review URL: https://chromiumcodereview.appspot.com/11266049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@164495 0039d316-1c4b-4281-b951-d872f2087c98
* Fix the failed cases in URLFetcherFileTest on Android.shouqun.liu@intel.com2012-10-251-8/+8
| | | | | | | | | | | | | * Push the needed data files to target device. * Set the correct document root. BUG= TEST=net_unittests --gtest_filter=URLFetcherFileTest.* Review URL: https://chromiumcodereview.appspot.com/10986042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@164061 0039d316-1c4b-4281-b951-d872f2087c98
* net: Remove unused URLRequest::job()hashimoto@chromium.org2012-10-251-4/+0
| | | | | | | | | | | | It's good timing to delete this method since it's unused. BUG=None TEST=build Review URL: https://chromiumcodereview.appspot.com/11260013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@164017 0039d316-1c4b-4281-b951-d872f2087c98
* Coalesce payload length statistics in ChromeNetworkDelegatebengr@google.com2012-10-243-2/+15
| | | | | | | | | BUG= Review URL: https://chromiumcodereview.appspot.com/11137022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163956 0039d316-1c4b-4281-b951-d872f2087c98
* Non-functional change to simply have Downloads query Prefs for the charset ↵pauljensen@chromium.org2012-10-232-9/+0
| | | | | | | | | | when we need it rather than have ChromeURLRequestContext track it on the network thread and pass it through three different classes. BUG=146596 Review URL: https://chromiumcodereview.appspot.com/11092088 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163652 0039d316-1c4b-4281-b951-d872f2087c98
* [net] Change factory methods for HostResolver and HostCache to return a ↵szym@chromium.org2012-10-225-38/+14
| | | | | | | | | | | | 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
* Make original_response_headers parameter of ↵battre@chromium.org2012-10-204-5/+5
| | | | | | | | | | | NetworkDelegate::OnHeadersReceived const BUG=149890 TBR=tony@chromium.org,mnaganov@chromium.org,jochen@chromium.org Review URL: https://chromiumcodereview.appspot.com/10944013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163111 0039d316-1c4b-4281-b951-d872f2087c98
* net: Make UploadDataStream::Read() asynchronoushashimoto@chromium.org2012-10-171-1/+2
| | | | | | | | | | | | | | Rename existing Read() to ReadSync() Change type of |buf| from char* to scoped_refptr<IOBuffer> so that async operation is always performed safely Add an asynchronous implementation, Read() The newly added Read() is not used yet, all users still use the old version ReadSync() BUG=72001 TEST=net_unittests Review URL: https://chromiumcodereview.appspot.com/10910268 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162343 0039d316-1c4b-4281-b951-d872f2087c98
* Use ScopedCOMInitializer in more places. While this doesn't always simplify ↵pkasting@chromium.org2012-10-031-2/+2
| | | | | | | | | | code, it does mean we do consistent logging and error-checking at all these sites. BUG=none TEST=none Review URL: https://codereview.chromium.org/11050009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@159908 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 146616.erikwright@chromium.org2012-10-022-5/+0
| | | | | | | | | | | | | | | | | | | | Account for server vs host clock skew in cookie expiration times. When setting a cookie's expiration time in the cookie store we need to take into account any difference between the HTTP server and the host machine's real time clock. BUG=135131 TEST=net_unittests --gtest_filter=CookieMonster/CookieStoreTest/0.TestCookieDeletion Review URL: https://chromiumcodereview.appspot.com/10692137 R=pauljensen@chromium.org,szym@chromium.org BUG=152078 Review URL: https://chromiumcodereview.appspot.com/11036007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@159685 0039d316-1c4b-4281-b951-d872f2087c98
* Miscellaneous tiny cleanups done while converting files to use ↵pkasting@chromium.org2012-10-011-17/+11
| | | | | | | | | | ScopedCOMInitializer, pulled out separately to make review easier. BUG=none TEST=none Review URL: https://codereview.chromium.org/10991052 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@159526 0039d316-1c4b-4281-b951-d872f2087c98
* URLRequestHttpJob::StartTransaction should honour network delegate.vabr@chromium.org2012-09-273-14/+72
| | | | | | | | | | | If a network delegate returns net::ERR_BLOCKED_BY_CLIENT inside URLRequestHttpJob::StartTransaction() the job is not cancelled. This does not happen, e.g., when ERR_BLOCKED_BY_CLIENT is returned within URLRequestHttpJob::OnStartCompleted(). This CL attempts to change the behaviour of StartTransaction() to cancel the job on the error signal from the delegate. BUG=146816 TEST=This is not testable in the browser without custom code changes yet. Review URL: https://chromiumcodereview.appspot.com/10911151 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158984 0039d316-1c4b-4281-b951-d872f2087c98
* There are two blocking network delegates in url_request_unittest:vabr@chromium.org2012-09-262-211/+352
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BlockingNetworkDelegate and BlockingNetworkDelegateWithManualCallback They provide various functions: * blocking in various stages of the request * three ways of responding in blocked stages: synchronous, automatic callback, and user-triggered callback * redirection But not all combinations are available (e.g. synchronous blocking after headers are sent), because the user has to chose only one of the delegates. This CL aims at allowing to combine those functions without code duplication, by joining the two classes into one. BUG=146816 TEST=N/A (this is only in unittests, also no change in behaviour of any unit-test, and no new added) Historical description for this issue, remove once the current plan stabilises: ------------------------------------------------------ Adding a base class for the blocking network delegates There are two blocking network delegates in url_request_unittest: BlockingNetworkDelegate and BlockingNetworkDelegateWithManualCallback They were independent until now, but for http://codereview.chromium.org/10911151/ BlockingNetworkDelegate needs to be able to block in various stages of the request, such as BlockingNetworkDelegateWithManualCallback does. To avoid code duplication, the common functionality is here moved to a common base class BlockingNetworkDelegateBase from which the original two inherit. This adds no new functions, nor changes it the behaviour of the current tests. ------------------------------------------------------ Review URL: https://chromiumcodereview.appspot.com/10905259 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158724 0039d316-1c4b-4281-b951-d872f2087c98
* Tests were marked as Flaky.vitalybuka@chromium.org2012-09-231-2/+2
| | | | | | | | | | BUG=151811,151810 TBR=droger@chromium.org,shalev@chromium.org NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10968052 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158204 0039d316-1c4b-4281-b951-d872f2087c98
* SampleSet -> HistogramSamples which can be reused by SparseHistogramkaiwang@chromium.org2012-09-221-15/+24
| | | | | | | | BUG=139612 Review URL: https://chromiumcodereview.appspot.com/10829466 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158166 0039d316-1c4b-4281-b951-d872f2087c98
* URLFetcher uses a TaskRunner instead of a message loop.droger@chromium.org2012-09-197-47/+59
| | | | | | | | | | | | Note that with this change, the URLFetcher can still be used with a MessageLoop, since MessageLoop is a task runner. BUG=NONE Review URL: https://chromiumcodereview.appspot.com/10915307 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157545 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: avoid foo ? true : false, part 2.thestig@chromium.org2012-09-191-7/+6
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10942004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157509 0039d316-1c4b-4281-b951-d872f2087c98
* Move TestJobInterceptor to url_request_test_util.droger@chromium.org2012-09-183-35/+52
| | | | | | | | | | | TestJobInterceptor is now accessible to other tests. BUG=None Review URL: https://chromiumcodereview.appspot.com/10919317 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157345 0039d316-1c4b-4281-b951-d872f2087c98
* Reordering URLRequest unittest and port to iOS.droger@chromium.org2012-09-121-2201/+2206
| | | | | | | | | | | | This CL groups the tests by case, excludes FTP-related unittests when FTP support is disabled, and disables tests using the TestServer on iOS. BUG=145954 Review URL: https://chromiumcodereview.appspot.com/10914205 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156300 0039d316-1c4b-4281-b951-d872f2087c98
* Fix and re-commit http://codereview.chromium.org/10914109/ (after revert in ↵gab@chromium.org2012-09-121-1/+5
| | | | | | | | | | | | | | | | | | | | | | | http://crrev.com/155918) -- Refactoring and tests for the highly undertested file_util::CreateOrUpdateShortcutLink() method. Simplify file_util::CreateOrUpdateShortcutLink()'s interface (use a struct to set parameters passed which allows callers to specify exactly what they want without having to pass in a bunch of NULLs for the unused parameters). The same concept will be used for ShellUtil's shortcut functions in an upcoming CL. Moved ShellUtil::VerifyChromeShortcut() to file_util::VerifyShortcut() and augmented it for every shortcut properties. This will also allow other shortcut creators (web apps, profiles, etc.) to have a broader test coverage on the shortcut they create (i.e. more testable properties available). I will leave it up to the owners of these various projects to augment their tests, this CL keeps the previously tested behavior, not more, not less. This is the 1st CL of a massive refactoring effort for shortcuts (http://goo.gl/Az889) in which ShellUtil's shortcut methods have to be refactored (http://codereview.chromium.org/10836247/ : soon to incorporate interface changes from this CL) which led me even lower to first refactor file_util's shortcut methods. TBR=robertshield@chromium.org, sky@chromium.org, agl@chromium.org, dgrogan@chromium.org BUG=132825, 148539 TEST=base_unittests --gtest_filter=FileUtilShortcutTest* installer_util_unitests --gtest_filter=ShellUtilTestWithDirAndDist* unit_tests --gtest_filter=ProfileShortcutManagerTest* (run tests on XP as well) Review URL: https://chromiumcodereview.appspot.com/10909171 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156250 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 153131 (Histograms showed it doesn't help much)mmenke@chromium.org2012-09-113-209/+1
| | | | | | | | | | | | | | | | | | 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
* installer_util_unittests has failed on XP Tests (dbg)(1) since this landeddgrogan@chromium.org2012-09-111-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | | Revert 155869 - Refactoring and tests for the highly undertested file_util::CreateOrUpdateShortcutLink() method. Simplify file_util::CreateOrUpdateShortcutLink()'s interface (use a struct to set parameters passed which allows callers to specify exactly what they want without having to pass in a bunch of NULLs for the unused parameters). The same concept will be used for ShellUtil's shortcut functions in an upcoming CL. Moved ShellUtil::VerifyChromeShortcut() to file_util::VerifyShortcut() and augmented it for every shortcut properties. This will also allow other shortcut creators (web apps, profiles, etc.) to have a broader test coverage on the shortcut they create (i.e. more testable properties available). I will leave it up to the owners of these various projects to augment their tests, this CL keeps the previously tested behavior, not more, not less. This is the 1st CL of a massive refactoring effort for shortcuts (http://goo.gl/Az889) in which ShellUtil's shortcut methods have to be refactored (http://codereview.chromium.org/10836247/ : soon to incorporate interface changes from this CL) which led me even lower to first refactor file_util's shortcut methods. BUG=132825 TEST=base_unittests --gtest_filter=FileUtilShortcutTest* installer_util_unitests --gtest_filter=ShellUtilTestWithDirAndDist* unit_tests --gtest_filter=ProfileShortcutManagerTest* Review URL: https://chromiumcodereview.appspot.com/10914109 TBR=gab@chromium.org Review URL: https://chromiumcodereview.appspot.com/10910183 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155918 0039d316-1c4b-4281-b951-d872f2087c98
* Refactoring and tests for the highly undertested ↵gab@chromium.org2012-09-101-1/+5
| | | | | | | | | | | | | | | | | | | | | | file_util::CreateOrUpdateShortcutLink() method. Simplify file_util::CreateOrUpdateShortcutLink()'s interface (use a struct to set parameters passed which allows callers to specify exactly what they want without having to pass in a bunch of NULLs for the unused parameters). The same concept will be used for ShellUtil's shortcut functions in an upcoming CL. Moved ShellUtil::VerifyChromeShortcut() to file_util::VerifyShortcut() and augmented it for every shortcut properties. This will also allow other shortcut creators (web apps, profiles, etc.) to have a broader test coverage on the shortcut they create (i.e. more testable properties available). I will leave it up to the owners of these various projects to augment their tests, this CL keeps the previously tested behavior, not more, not less. This is the 1st CL of a massive refactoring effort for shortcuts (http://goo.gl/Az889) in which ShellUtil's shortcut methods have to be refactored (http://codereview.chromium.org/10836247/ : soon to incorporate interface changes from this CL) which led me even lower to first refactor file_util's shortcut methods. BUG=132825 TEST=base_unittests --gtest_filter=FileUtilShortcutTest* installer_util_unitests --gtest_filter=ShellUtilTestWithDirAndDist* unit_tests --gtest_filter=ProfileShortcutManagerTest* Review URL: https://chromiumcodereview.appspot.com/10914109 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155869 0039d316-1c4b-4281-b951-d872f2087c98
* Remove static variables from HttpStreamFactory.szager@chromium.org2012-09-074-27/+80
| | | | | | | | | | | Refresh of issue 10389162. BUG=124900 Review URL: https://chromiumcodereview.appspot.com/10834215 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155485 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the HANDLED_EXTERNALLY status code.mkosiba@chromium.org2012-09-072-8/+2
| | | | | | | | | | | | This removes the HANDLED_EXTERNALLY status replacing its use with ResourceRequestInfo::HandledExternally. BUG=none TEST=unit_tests, content_unittests Review URL: https://chromiumcodereview.appspot.com/10640019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155427 0039d316-1c4b-4281-b951-d872f2087c98
* Implement SHA-256 fingerprint supportpalmer@chromium.org2012-09-071-4/+4
| | | | | | | | | | | | | | | | The HTTP-based Public Key Pinning Internet Draft (tools.ietf.org/html/draft-ietf-websec-key-pinning) requires this. Per wtc, give the *Fingeprint* types more meaningful *HashValue* names. Cleaning up lint along the way. BUG=117914 TEST=net_unittests, unit_tests TransportSecurityPersisterTest Review URL: https://chromiumcodereview.appspot.com/10826257 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155365 0039d316-1c4b-4281-b951-d872f2087c98
* Log the source when a URLFetcher leak is detectedrsimha@chromium.org2012-09-061-1/+2
| | | | | | | | | | | | | | Now that we have a mechanism to detect URLFetcher leaks in test suites, we're seeing instances of leaks that are causing redness on the chromium waterfall. However, the logs do not reveal which URLFetcher caused the leak, and makes fixing the problem hard, particularly when the leak is a rarity. This patch adds logging info that specifies the source of the leak. TBR=willchan@chromium.org BUG=146826 TEST=Run a test that leaks a URL, and make sure the logs indicate the source. Review URL: https://chromiumcodereview.appspot.com/10914134 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155278 0039d316-1c4b-4281-b951-d872f2087c98
* Restrict file access on android.nileshagrawal@chromium.org2012-09-061-4/+0
| | | | | | | | | | | Only files in external storage directory are allowed. BUG= Review URL: https://chromiumcodereview.appspot.com/10905114 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155121 0039d316-1c4b-4281-b951-d872f2087c98
* Add get_server_time utilityakalin@chromium.org2012-09-051-0/+4
| | | | | | | | | | | | | | | | | | get_server_time makes a request to an HTTP/HTTPS server and snarfs the server time from the HTTP headers. The code here will eventually be adapted to track "sane time" (see design document linked from the bug). Also update usage comments of URLFetcher to note that a request context has to be set. BUG=146090 Review URL: https://chromiumcodereview.appspot.com/10907039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154868 0039d316-1c4b-4281-b951-d872f2087c98
* Stop using ScopedAllowIO in content::ResourceDispatcherHostImplhashimoto@chromium.org2012-08-309-18/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: UploadData::GetContentLengthSync() call is removed from ResourceDispatcherHostImpl. The return type of net::URLRequest::GetUploadProgress() is changed from uint64 to net::UploadProgress. A new member 'upload_size' is added to AutomationURLResponse for Chrome Frame. content::ResourceDispatcherHostImpl has been using UploadData::GetContentLengthSync() which needs ScopedAllowIO. To stop using ScopedAllowIO, there were three options considered. 1. Use asynchronous version UploadData::GetContentLength() which does not need ScopedAllowIO. pros: Changes would be minimal and straight-forward. cons: GetContentLength() is also called in UploadDataStream::Init(). If we start reusing the value, there is no need to call the same method here. 2. Communicate the upload data size to ResourceDispatcherHost by adding an event OnRequestBodyInitialized() to URLRequest::Delegate. pros: No duplicated GetContentLength() call here and in UploadDataStream::Init(). cons: Complicated interface. 3. Return upload data size as well as upload progress from URLRequest::GetUploadProgress(). pros: No duplicated GetContentLength() call here and in UploadDataStream::Init(). Simple interface. Making sense because upload progress and upload size are almost always used together (see ResourceHandler::OnUploadProgress and DOM ProgressEvent as examples). cons: Polling upload data size may imply that the size may change. We've decided to go with #3. The return type of net::URLRequest::GetUploadProgress() is changed from uint64 to net::UploadProgress, and URLRequest::GetUploadProgress() is used to get the upload size from ResourceDispatcherHostImpl instead of UploadData::GetContentLengthSync(). In Chrome Frame, URLRequestAutomationJob is used instead of URLRequestHttpJob and UploadDataStream is not initialized in the browser process. This is problematic since we cannot know the size of upload data without UploadDataStream. To deal with this, a new member 'upload_size' is added to AutomationURLResponse and the value is used to implement URLRequestAutomationJob::GetUploadProgress(). BUG=112607 TEST=Can upload files Review URL: https://chromiumcodereview.appspot.com/10825073 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154286 0039d316-1c4b-4281-b951-d872f2087c98
* Disable FTP on iOSdroger@chromium.org2012-08-304-1/+17
| | | | | | | | | | | FTP is not supported on iOS. This is not obvious, because net.gyp builds almost nothing on iOS currently, but we plan on keeping the ftp/ directory excluded from iOS builds. BUG=145459 Review URL: https://chromiumcodereview.appspot.com/10880071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154130 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
* Added URLRequestContext::CreateRequest which can be used to create URLRequestsshalev@chromium.org2012-08-245-31/+86
| | | | | | | | | BUG=142945 Review URL: https://chromiumcodereview.appspot.com/10873056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153302 0039d316-1c4b-4281-b951-d872f2087c98
* Turned job_factory into a pure virtual classshalev@chromium.org2012-08-247-153/+219
| | | | | | | | | | | | This makes it possible to create wrappers for job_factory that handle a particular job type, and forward the rest to the old job_factory. TBR=michaeln@chromium.org, brettw@chromium.org, davemoore@chromium.org, kalman@chromium.org, benjhayden@chromium.org, jochen@chromium.org BUG=None Review URL: https://chromiumcodereview.appspot.com/10836248 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153241 0039d316-1c4b-4281-b951-d872f2087c98
* Refactoring: ProtocolHandler::MaybeCreateJob and other functions take ↵shalev@chromium.org2012-08-2438-197/+267
| | | | | | | | | | | | | | NetworkDelegate as argument This change goes a long way to prepare for removing NetworkDelegate from URLRequestContext. TBR=sky@chromium.org, michaeln@chromium.org, benjhayden@chromium.org, brettw@chromium.org, ben@chromium.org, davemoore@chromium.org, zelidrag@chromium.org, mnissler@chromium.org, thestig@chromium.org, asargent@chromium.org, jhawkins@chromium.org, bulach@chromium.org BUG=crbug.com/142945 Review URL: https://chromiumcodereview.appspot.com/10855209 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153133 0039d316-1c4b-4281-b951-d872f2087c98
* Reland of http://codereview.chromium.org/10854204/, whichmmenke@chromium.org2012-08-243-1/+209
| | | | | | | | | | | | | 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-233-209/+1
| | | | | | | | | | | | | | 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-233-1/+209
| | | | | | | | | | | 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
* Refactoring for merging WebSocket test server to net::TestServertoyoshim@chromium.org2012-08-232-70/+82
| | | | | | | | | | | | - rename HTTPSOptions to SSLOptions - pass an argument for secure server to specify the service protocol type BUG=137639 Review URL: https://chromiumcodereview.appspot.com/10878003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152902 0039d316-1c4b-4281-b951-d872f2087c98
* When trying to identify when network data is received while thepauljensen@chromium.org2012-08-211-1/+2
| | | | | | | | | | | | NetworkChangeNotifier thinks we are offline, ignore cases when there is no data received or the data came from the cache. BUG=124069 Review URL: https://chromiumcodereview.appspot.com/10860045 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152483 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 151198 - Turn off TLS 1.1.wtc@chromium.org2012-08-201-0/+4
| | | | | | | | | | | | | | | | Review URL: https://chromiumcodereview.appspot.com/10828272 Bug 141629 has been fixed. We can turn on TLS 1.1 on the trunk. TBR=agl@chromium.org,rsleevi@chromium.org BUG=142172 TEST=Visit https://www.google.com/ and https://www.facebook.com/. Click the lock icon. The page info bubble should say "The connection uses TLS 1.1." Review URL: https://chromiumcodereview.appspot.com/10854212 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152404 0039d316-1c4b-4281-b951-d872f2087c98
* Add histograms for network activity, and total/cumulativetburkard@chromium.org2012-08-204-13/+44
| | | | | | | | histograms for cache/network activity. R=rvargas@chromium.org, mmenke@chromium.org Review URL: https://chromiumcodereview.appspot.com/10834313 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152398 0039d316-1c4b-4281-b951-d872f2087c98
* Replaced static URLRequestFileJob factory with non-static protocol handler ↵shalev@chromium.org2012-08-204-20/+93
| | | | | | | | | | | | | for File jobs This is a refactor and shouldn't have any visible behavioral change. BUG=crbug.com/142945 TEST=browser_tests --single_process --gtest_filter=FullscreenControllerTest.FullscreenFileURL Review URL: https://chromiumcodereview.appspot.com/10700117 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152381 0039d316-1c4b-4281-b951-d872f2087c98
* net: Fix more clang warnings about missing virtual and OVERRIDE annotations.tfarina@chromium.org2012-08-191-9/+9
| | | | | | | | | | BUG=115047 R=willchan@chromium.org Review URL: https://chromiumcodereview.appspot.com/10825437 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152296 0039d316-1c4b-4281-b951-d872f2087c98