Add control to use If-Range header for range request.

content/browser/download/download_worker.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/download_worker.h"
 
 #include "content/browser/download/download_create_info.h"
 #include "content/public/browser/download_interrupt_reasons.h"
 #include "content/public/browser/web_contents.h"
 
@@ skipping 71 matching lines @@
   // |callback| is not used in subsequent requests.
   DCHECK(callback.is_null());
 
   // Destroy the request if user canceled.
   if (is_canceled_) {
     VLOG(kVerboseLevel) << "Byte stream arrived after user cancel the request.";
     create_info->request_handle->CancelRequest();
     return;
   }
 
-  // TODO(xingliu): Add the interrupt reason and metric data for precondition
-  // failure. Make DownloadRequestCore know if it should return error if the
-  // the server gives a different part of the content, e.g. "If-Match" return
-  // http 200.
+  // TODO(xingliu): Add metric for error handling.
   if (create_info->result !=
       DownloadInterruptReason::DOWNLOAD_INTERRUPT_REASON_NONE) {
     VLOG(kVerboseLevel) << "Parallel download sub-request failed. reason = "
                         << create_info->result;
+
+    // Ignore HTTP 416 for the workers.
+    if (create_info->result ==

[qinmin, 2017/03/30 05:21:43]

{} needed as this if statement spans multiple lines

[xingliu, 2017/03/30 17:16:46]

Done.

+        DownloadInterruptReason::DOWNLOAD_INTERRUPT_REASON_SERVER_NO_RANGE)
+      return;
+
     delegate_->OnServerResponseError(this, create_info->result);
     return;
   }
 
   request_handle_ = std::move(create_info->request_handle);
 
   // Pause the stream if user paused, still push the stream reader to the sink.
   if (is_paused_) {
     VLOG(kVerboseLevel) << "Byte stream arrived after user pause the request.";
     Pause();
   }
 
   delegate_->OnByteStreamReady(this, std::move(stream_reader));
 }
 
 void DownloadWorker::OnUrlDownloaderStopped(UrlDownloader* downloader) {
   // Release the |url_downloader_|, the object will be deleted on IO thread.
   url_downloader_.reset();
 }
 
 void DownloadWorker::AddUrlDownloader(
     std::unique_ptr<UrlDownloader, BrowserThread::DeleteOnIOThread>
         downloader) {
   url_downloader_ = std::move(downloader);
 }
 
 }  // namespace content