Added member data to classes to support download resumption.

content/browser/download/download_persistent_store_info.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.
 //
 // Download struct used for informing and querying the download system's
 // persitent store.
 
 #ifndef CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_PERSISTENT_STORE_INFO_H_
 #define CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_PERSISTENT_STORE_INFO_H_
 #pragma once
 
+#include <string>
 #include <vector>
 
 #include "base/file_path.h"
 #include "base/time.h"
+#include "content/browser/download/interrupt_reasons.h"
 #include "content/common/content_export.h"
 #include "googleurl/src/gurl.h"
 
 class DownloadItem;
 
 // Contains the information that is stored in the download system's persistent
 // store (or refers to it).  Managed by the DownloadItem.  When used to create a
 // history entry, all fields except for |db_handle| are set by DownloadItem and
 // read by DownloadDatabase.  When used to update a history entry, DownloadItem
 // sets all fields, but DownloadDatabase only reads |end_time|,
 // |received_bytes|, |state|, and |opened|, and uses |db_handle| to select the
 // row in the database table to update.  When used to load DownloadItems from
 // the history, all fields except |referrer_url| are set by the DownloadDatabase
 // and read by the DownloadItem.
 struct CONTENT_EXPORT DownloadPersistentStoreInfo {
-  // TODO(ahendrickson) -- Reduce the number of constructors.
   DownloadPersistentStoreInfo();
   DownloadPersistentStoreInfo(const FilePath& path,
                               const GURL& url,
                               const GURL& referrer,
                               const base::Time& start,
                               const base::Time& end,
                               int64 received,
                               int64 total,
                               int32 download_state,
                               int64 handle,
-                              bool download_opened);
+                              bool download_opened,
+                              int32 local_id,
+                              const std::string& hash,
+                              const std::string& last_modified,
+                              const std::string& etag,
+                              InterruptReason reason);
   ~DownloadPersistentStoreInfo();  // For linux-clang.
 
   // The final path where the download is saved.
   FilePath path;
 
   // The URL from which we are downloading. This is the final URL after any
   // redirection by the server for |url_chain|. Is not changed by UpdateEntry().
   GURL url;
 
   // The URL that referred us.
@@ skipping 13 matching lines @@
 
   // The current state of the download.
   int32 state;
 
   // The handle of the download in the database. Is not changed by
   // UpdateEntry().
   int64 db_handle;
 
   // Whether this download has ever been opened from the browser.
   bool opened;
+
+  // The local ID of the download (unique per profile, and therefore per
+  // history database).
+  int32 local_id;

[Randy Smith (Not in Mondays), 2011/11/16 18:29:27]

Why do we want to persist this?  It's namespace is per-browser instance, and
when that stops being true, we'll have unified it with the db_handle.

[ahendrickson, 2011/11/19 20:18:03]

Removed.

+
+  // Sha256 hash of the content.  This might be empty either because
+  // the download hasn't started yet or because the hash isn't needed
+  // (ChromeDownloadManagerDelegate::GenerateFileHash() returned false).
+  std::string hash;
+
+  // Server's time stamp for the file.
+  std::string last_modified_time;
+
+  // Server's ETAG for the file.
+  std::string etag;
+
+  // Last reason.

[Randy Smith (Not in Mondays), 2011/11/16 18:29:27]

A bit more of a comment here?  Duplicating the variable name doesn't really feel
like it deserves the space taken by the comment :-}.

[ahendrickson, 2011/11/19 20:18:03]

Done.

+  InterruptReason last_reason;
 };
 
 #endif  // CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_PERSISTENT_STORE_INFO_H_