Add support for OfflinePagesDownloadBridge.

components/offline_pages/downloads/download_ui_item.h

Index: components/offline_pages/downloads/download_ui_item.h
diff --git a/components/offline_pages/downloads/download_ui_item.h b/components/offline_pages/downloads/download_ui_item.h
new file mode 100644
index 0000000000000000000000000000000000000000..3ae71a70ff368d65b78b02d77dedd04f868b9f44
--- /dev/null
+++ b/components/offline_pages/downloads/download_ui_item.h
@@ -0,0 +1,44 @@
+// Copyright 2016 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 COMPONENTS_OFFLINE_PAGES_DOWNLOADS_DOWNLOAD_UI_ITEM_H_
+#define COMPONENTS_OFFLINE_PAGES_DOWNLOADS_DOWNLOAD_UI_ITEM_H_
+
+#include <string>
+
+#include "base/files/file_path.h"
+#include "base/time/time.h"
+#include "components/offline_pages/offline_page_item.h"
+#include "url/gurl.h"
+
+namespace offline_pages {
+
+struct DownloadUIItem {
+ public:
+  DownloadUIItem();
+  explicit DownloadUIItem(const OfflinePageItem& page);
+  ~DownloadUIItem();
+
+  // Unique id.
+  std::string guid;
+
+  // The URL of the captured page.
+  GURL url;
+
+  // The file path to the archive with a local copy of the page.
+  base::FilePath target_path;

[fgorski, 2016/07/25 17:09:57]

having an extra method: GetOfflineURL() might be helpful to add (now or later)
for cases where we want to use download item to launch offline page.

[Dmitry Titov, 2016/07/26 00:41:25]

I thought UI won't launch anything, instead they will pass "open" command to us
and we'll open or whatever semantically is needed for the download item...

[fgorski, 2016/07/26 16:25:21]

Acknowledged.

+
+  // Most often "multipart/related".
+  std::string mime_type;

[fgorski, 2016/07/25 17:09:57]

Based on comment from Dan in my code, we don't need that for offline page items.

[Dmitry Titov, 2016/07/26 00:41:25]

Removed, we might need it later (to be a package).

+
+  // The time when the offline archive was created.
+  base::Time start_time_ms;

[fgorski, 2016/07/25 17:09:57]

if it is a base::Time, then _ms is not appropriate here.

[Dmitry Titov, 2016/07/26 00:41:25]

Done.

+
+  // The size of the offline copy.
+  int64_t total_bytes;
+};
+
+}  // namespace offline_pages
+
+#endif  // COMPONENTS_OFFLINE_PAGES_DOWNLOADS_DOWNLOAD_UI_ITEM_H_