Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(348)

Unified Diff: components/offline_pages/downloads/download_ui_item.h

Issue 2260963003: [Offline pages] Enabling notifications for pages saved later (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Adding tests Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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
index 19c31c544431cdf25fe701693417e94721ea4967..801c1ef8e10f719e0857f313e6f64627dd9b1c33 100644
--- a/components/offline_pages/downloads/download_ui_item.h
+++ b/components/offline_pages/downloads/download_ui_item.h
@@ -11,15 +11,18 @@
#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 OfflinePageItem;
+class SavePageRequest;
+
struct DownloadUIItem {
public:
DownloadUIItem();
explicit DownloadUIItem(const OfflinePageItem& page);
+ explicit DownloadUIItem(const SavePageRequest& request);
Dmitry Titov 2016/08/24 00:29:36 I wonder if we need a bool is_downloaded on the it
fgorski 2016/08/24 18:25:23 Acknowledged. Let's add that later if/when we need
DownloadUIItem(const DownloadUIItem& other);
~DownloadUIItem();

Powered by Google App Engine
This is Rietveld 408576698