diff options
| author | benjhayden@chromium.org <benjhayden@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-12 18:22:43 +0000 |
|---|---|---|
| committer | benjhayden@chromium.org <benjhayden@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-12 18:22:43 +0000 |
| commit | 2c1b5cbf57527974ca6a8291029b5e70b5ee0649 (patch) | |
| tree | 25057d3aa55456b3b7cdf91129511ea04491e5a1 /content/browser/download/download_resource_handler.cc | |
| parent | 9a065829eb38d8f47482d353368c244a67729618 (diff) | |
| download | chromium_src-2c1b5cbf57527974ca6a8291029b5e70b5ee0649.zip chromium_src-2c1b5cbf57527974ca6a8291029b5e70b5ee0649.tar.gz chromium_src-2c1b5cbf57527974ca6a8291029b5e70b5ee0649.tar.bz2 | |
DownloadResourceHandler should consider more 2XX codes to be success.
BUG=248875
Review URL: https://chromiumcodereview.appspot.com/16663007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205877 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/download/download_resource_handler.cc')
| -rw-r--r-- | content/browser/download/download_resource_handler.cc | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/content/browser/download/download_resource_handler.cc b/content/browser/download/download_resource_handler.cc index a2c3487..c2a2438 100644 --- a/content/browser/download/download_resource_handler.cc +++ b/content/browser/download/download_resource_handler.cc @@ -347,6 +347,10 @@ bool DownloadResourceHandler::OnResponseCompleted( switch(response_code) { case -1: // Non-HTTP request. case net::HTTP_OK: + case net::HTTP_CREATED: + case net::HTTP_ACCEPTED: + case net::HTTP_NON_AUTHORITATIVE_INFORMATION: + case net::HTTP_RESET_CONTENT: case net::HTTP_PARTIAL_CONTENT: // Expected successful codes. break; @@ -365,12 +369,10 @@ bool DownloadResourceHandler::OnResponseCompleted( reason = DOWNLOAD_INTERRUPT_REASON_SERVER_NO_RANGE; break; default: // All other errors. - // Redirection should have been handled earlier in the stack. - DCHECK(3 != response_code / 100); - - // Informational codes should have been handled earlier in the - // stack. - DCHECK(1 != response_code / 100); + // Redirection and informational codes should have been handled earlier + // in the stack. + DCHECK_NE(3, response_code / 100); + DCHECK_NE(1, response_code / 100); reason = DOWNLOAD_INTERRUPT_REASON_SERVER_FAILED; break; } |