Added member data to classes to support download resumption.

content/browser/download/download_types.h

 // Copyright (c) 2011 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_TYPES_H_
 #define CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_TYPES_H_
 #pragma once
 
 #include "base/file_path.h"
 #include "base/memory/linked_ptr.h"
 #include "content/common/content_export.h"
 #include "net/base/file_stream.h"
 
 // Holds the information about how to save a download file.

[Randy Smith (Not in Mondays), 2011/11/22 17:37:49]

I think it makes sense to have a comment here something like "In the case of
download resumption, file_path is set to ... , offset ... hash_state ..." for
context.  I'd very much like to have the interface (the contents of the
structure) be independent of resumption and flexible, but given the context will
probably help people understand the "why" of the new items.

[ahendrickson, 2011/11/22 23:46:44]

Done.

 struct CONTENT_EXPORT DownloadSaveInfo {
   DownloadSaveInfo();
   DownloadSaveInfo(const DownloadSaveInfo& info);
   ~DownloadSaveInfo();
   DownloadSaveInfo& operator=(const DownloadSaveInfo& info);
 
+  // This is usually the tentative final name, but not during resumption.

[Randy Smith (Not in Mondays), 2011/11/22 17:37:49]

nit: Specify what it is during resumption.

[ahendrickson, 2011/11/22 23:46:44]

Done.

   FilePath file_path;
+
   linked_ptr<net::FileStream> file_stream;
+
   string16 suggested_name;
+
+  // The file offset at which to start the download.  May be 0.
+  size_t offset;
+
+  // The state of the hash at the start of the download.  May be empty.
+  std::string hash_state;
 };
 
 #endif  // CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_TYPES_H_