Make DownloadHistory observe manager, items

chrome/browser/history/download_persistent_store_info.h

Index: chrome/browser/history/download_persistent_store_info.h
diff --git a/chrome/browser/history/download_persistent_store_info.h b/chrome/browser/history/download_persistent_store_info.h
new file mode 100644
index 0000000000000000000000000000000000000000..1971ce2852480d942cad004faaaeef7a73eee3da
--- /dev/null
+++ b/chrome/browser/history/download_persistent_store_info.h
@@ -0,0 +1,64 @@
+// 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 CHROME_BROWSER_HISTORY_DOWNLOAD_PERSISTENT_STORE_INFO_H_
+#define CHROME_BROWSER_HISTORY_DOWNLOAD_PERSISTENT_STORE_INFO_H_
+
+#include "base/file_path.h"
+#include "base/time.h"
+#include "content/public/browser/download_item.h"
+#include "googleurl/src/gurl.h"
+
+// Contains the information that is stored in the download system's persistent
+// store (or refers to it). DownloadHistory uses this to communicate with the
+// DownloadDatabase through the HistoryService.
+struct DownloadPersistentStoreInfo {
+  DownloadPersistentStoreInfo();
+  DownloadPersistentStoreInfo(
+      const FilePath& path,
+      const GURL& url,
+      const GURL& referrer,
+      const base::Time& start,
+      const base::Time& end,
+      int64 received,
+      int64 total,
+      content::DownloadItem::DownloadState download_state,
+      int64 handle,
+      bool download_opened);
+  ~DownloadPersistentStoreInfo();
+
+  // The final path where the download is saved.

[asanka, 2012/09/24 18:39:43]

This comment isn't accurate. This could be the intermediate path.

[benjhayden, 2012/09/24 20:12:11]

Done.

+  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.

[asanka, 2012/09/24 18:39:43]

Nit: Note that it isn't changed?

[benjhayden, 2012/09/24 20:12:11]

Done.

+  GURL referrer_url;
+
+  // The time when the download started. Is not changed by UpdateEntry().
+  base::Time start_time;
+
+  // The time when the download completed.
+  base::Time end_time;
+
+  // The number of bytes received (so far).
+  int64 received_bytes;
+
+  // The total number of bytes in the download. Is not changed by UpdateEntry().
+  int64 total_bytes;
+
+  // The current state of the download.
+  content::DownloadItem::DownloadState 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;
+};
+
+#endif  // CHROME_BROWSER_HISTORY_DOWNLOAD_PERSISTENT_STORE_INFO_H_