Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(382)

Side by Side Diff: content/browser/download/download_item_impl.cc

Issue 1544603003: [Downloads] Do not store error responses during resumption. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@unify-downloader-core
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // File method ordering: Methods in this file are in the same order as 5 // File method ordering: Methods in this file are in the same order as
6 // in download_item_impl.h, with the following exception: The public 6 // in download_item_impl.h, with the following exception: The public
7 // interface Start is placed in chronological order with the other 7 // interface Start is placed in chronological order with the other
8 // (private) routines that together define a DownloadItem's state 8 // (private) routines that together define a DownloadItem's state
9 // transitions as the download progresses. See "Download progression 9 // transitions as the download progresses. See "Download progression
10 // cascade" later in this file. 10 // cascade" later in this file.
(...skipping 898 matching lines...) Expand 10 before | Expand all | Expand 10 after
909 if (force_user) 909 if (force_user)
910 mode = RESUME_MODE_USER_RESTART; 910 mode = RESUME_MODE_USER_RESTART;
911 else 911 else
912 mode = RESUME_MODE_IMMEDIATE_RESTART; 912 mode = RESUME_MODE_IMMEDIATE_RESTART;
913 break; 913 break;
914 914
915 case DOWNLOAD_INTERRUPT_REASON_NETWORK_FAILED: 915 case DOWNLOAD_INTERRUPT_REASON_NETWORK_FAILED:
916 case DOWNLOAD_INTERRUPT_REASON_NETWORK_DISCONNECTED: 916 case DOWNLOAD_INTERRUPT_REASON_NETWORK_DISCONNECTED:
917 case DOWNLOAD_INTERRUPT_REASON_NETWORK_SERVER_DOWN: 917 case DOWNLOAD_INTERRUPT_REASON_NETWORK_SERVER_DOWN:
918 case DOWNLOAD_INTERRUPT_REASON_SERVER_FAILED: 918 case DOWNLOAD_INTERRUPT_REASON_SERVER_FAILED:
919 case DOWNLOAD_INTERRUPT_REASON_SERVER_UNREACHABLE:
919 case DOWNLOAD_INTERRUPT_REASON_USER_SHUTDOWN: 920 case DOWNLOAD_INTERRUPT_REASON_USER_SHUTDOWN:
920 case DOWNLOAD_INTERRUPT_REASON_CRASH: 921 case DOWNLOAD_INTERRUPT_REASON_CRASH:
921 if (force_restart) 922 if (force_restart)
922 mode = RESUME_MODE_USER_RESTART; 923 mode = RESUME_MODE_USER_RESTART;
923 else 924 else
924 mode = RESUME_MODE_USER_CONTINUE; 925 mode = RESUME_MODE_USER_CONTINUE;
925 break; 926 break;
926 927
927 case DOWNLOAD_INTERRUPT_REASON_FILE_FAILED: 928 case DOWNLOAD_INTERRUPT_REASON_FILE_FAILED:
928 case DOWNLOAD_INTERRUPT_REASON_FILE_ACCESS_DENIED: 929 case DOWNLOAD_INTERRUPT_REASON_FILE_ACCESS_DENIED:
(...skipping 1096 matching lines...) Expand 10 before | Expand all | Expand 10 after
2025 case RESUME_MODE_USER_CONTINUE: 2026 case RESUME_MODE_USER_CONTINUE:
2026 return "USER_CONTINUE"; 2027 return "USER_CONTINUE";
2027 case RESUME_MODE_USER_RESTART: 2028 case RESUME_MODE_USER_RESTART:
2028 return "USER_RESTART"; 2029 return "USER_RESTART";
2029 } 2030 }
2030 NOTREACHED() << "Unknown resume mode " << mode; 2031 NOTREACHED() << "Unknown resume mode " << mode;
2031 return "unknown"; 2032 return "unknown";
2032 } 2033 }
2033 2034
2034 } // namespace content 2035 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/download/download_browsertest.cc ('k') | content/browser/download/download_request_core.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698