Reduce unnecessary download interruptions due to parallel requests

content/browser/download/parallel_download_job.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/download/parallel_download_job.h"
 
 #include <algorithm>
 
 #include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
@@ skipping 110 matching lines @@
   RecordParallelDownloadAddStreamSuccess(success);
 
   // Destroy the request if the sink is gone.
   if (!success) {
     VLOG(kVerboseLevel)
         << "Byte stream arrived after download file is released.";
     worker->Cancel();
   }
 }
 
-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());
-    return;
-  }
-  DownloadJob::Interrupt(reason);
-}
-
 void ParallelDownloadJob::BuildParallelRequests() {
   DCHECK(!requests_sent_);
   DCHECK(!is_paused());
   if (is_canceled_)
     return;
 
   // TODO(qinmin): The size of |slices_to_download| should be no larger than
   // |kParallelRequestCount| unless |kParallelRequestCount| is changed after
   // a download is interrupted. This could happen if we use finch to config
   // the number of parallel requests.
@@ skipping 87 matching lines @@
   // download request.
   download_params->set_referrer(Referrer(download_item_->GetReferrerUrl(),
                                          blink::kWebReferrerPolicyAlways));
   // Send the request.
   worker->SendRequest(std::move(download_params));
   DCHECK(workers_.find(offset) == workers_.end());
   workers_[offset] = std::move(worker);
 }
 
 }  // namespace content