Reduce unnecessary download interruptions due to parallel requests

content/browser/download/parallel_download_job.cc

Index: content/browser/download/parallel_download_job.cc
diff --git a/content/browser/download/parallel_download_job.cc b/content/browser/download/parallel_download_job.cc
index 5f282c3536dc091899a7ad58f328480ef4cc4f8f..e83df49104f09f0bc824521a7f51cf6a5268da14 100644
--- a/content/browser/download/parallel_download_job.cc
+++ b/content/browser/download/parallel_download_job.cc
@@ -128,21 +128,6 @@ void ParallelDownloadJob::OnByteStreamReady(
   }
 }
 
-void ParallelDownloadJob::OnServerResponseError(
-    DownloadWorker* worker,
-    DownloadInterruptReason reason) {
-  // TODO(xingliu): Consider to let the original request to cover the full
-  // content if the sub-requests get invalid response. Consider retry on certain
-  // error.
-  if (worker->length() == DownloadSaveInfo::kLengthFullContent &&
-      reason ==
-          DownloadInterruptReason::DOWNLOAD_INTERRUPT_REASON_SERVER_NO_RANGE) {
-    SetPotentialFileLength(worker->offset());

[xingliu, 2017/05/10 01:40:17]

nit: Shall we remove DownloadJob::SetPotentialFileLength and
DownloadJob::Interrupt? since all errors now go through DownloadFile.

[qinmin, 2017/05/10 17:47:24]

Done.

-    return;
-  }
-  DownloadJob::Interrupt(reason);
-}
-
 void ParallelDownloadJob::BuildParallelRequests() {
   DCHECK(!requests_sent_);
   DCHECK(!is_paused());