[Offline pages] Enabling notifications for pages saved later

components/offline_pages/downloads/download_ui_item.h

 // 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 <stdint.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 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 item, and initialize it one way
or another depending which ctor is used. Of course only if it's useflu in UI, so
can add later when we need it.

[fgorski, 2016/08/24 18:25:23]

Acknowledged. Let's add that later if/when we need it.

   DownloadUIItem(const DownloadUIItem& other);
   ~DownloadUIItem();
 
   // Unique id.
   std::string guid;
 
   // The URL of the captured page.
   GURL url;
 
   // The Title of the captured page, if any. It can be empty string either
   // because the page is not yet fully loaded, or because it doesn't have any.
   base::string16 title;
 
   // The file path to the archive with a local copy of the page.
   base::FilePath target_path;
 
   // The time when the offline archive was created.
   base::Time start_time;
 
   // The size of the offline copy.
   int64_t total_bytes;
 };
 
 }  // namespace offline_pages
 
 #endif  // COMPONENTS_OFFLINE_PAGES_DOWNLOADS_DOWNLOAD_UI_ITEM_H_