Glue parallel download job and download file together.

content/browser/download/download_create_info.h

 // Copyright (c) 2012 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.
 
 #ifndef CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_CREATE_INFO_H_
 #define CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_CREATE_INFO_H_
 
 #include <stdint.h>
 
 #include <string>
@@ skipping 42 matching lines @@
   // The URL of the tab that started us.
   GURL tab_url;
 
   // The referrer URL of the tab that started us.
   GURL tab_referrer_url;
 
   // The time when the download started.
   base::Time start_time;
 
   // The total download size.
   int64_t total_bytes;

[asanka, 2017/03/17 17:10:09]

This is now the size of the response body, not the total download size.

[xingliu, 2017/03/17 19:08:25]

Done, change the comment here, to address this is the content-length response
header.

 
+  // The starting position of the request to save to the target file.

[asanka, 2017/03/17 17:10:09]

Can you add a comment indicating that this must match save_info->offset if
save_info is non-null?

[xingliu, 2017/03/17 19:08:25]

Done, also put a TODO, I think if we refactor job to own download file, this can
be removed.

+  int64_t offset;
+
   // True if the download was initiated by user action.
   bool has_user_gesture;
 
   base::Optional<ui::PageTransition> transition_type;
 
   // The remote IP address where the download was fetched from.  Copied from
   // UrlRequest::GetSocketAddress().
   std::string remote_address;
 
   // If the download is initially created in an interrupted state (because the
@@ skipping 38 matching lines @@
   // If "Accept-Ranges:bytes" header presents in the response header.
   bool accept_range;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(DownloadCreateInfo);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_CREATE_INFO_H_