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

Unified Diff: ios/chrome/browser/reading_list/reading_list_download_service.h

Issue 2666643002: Remove usage of DomDistillerService in ReadingListDownloadService. (Closed)
Patch Set: missing JS Created 3 years, 11 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: ios/chrome/browser/reading_list/reading_list_download_service.h
diff --git a/ios/chrome/browser/reading_list/reading_list_download_service.h b/ios/chrome/browser/reading_list/reading_list_download_service.h
index bbc1c8fd3298d957aebe7948c97f97f584a9cc9a..92d0239530c68dff034ce0ae33352277e22a00d1 100644
--- a/ios/chrome/browser/reading_list/reading_list_download_service.h
+++ b/ios/chrome/browser/reading_list/reading_list_download_service.h
@@ -18,10 +18,6 @@ namespace base {
class FilePath;
}
-namespace dom_distiller {
-class DomDistillerService;
-}
-
namespace reading_list {
class ReadingListDistillerPageFactory;
}
@@ -37,10 +33,10 @@ class ReadingListDownloadService
public:
ReadingListDownloadService(
ReadingListModel* reading_list_model,
- dom_distiller::DomDistillerService* distiller_service,
PrefService* prefs,
base::FilePath chrome_profile_path,
net::URLRequestContextGetter* url_request_context_getter,
+ std::unique_ptr<dom_distiller::DistillerFactory> distiller_factory,
std::unique_ptr<reading_list::ReadingListDistillerPageFactory>
distiller_page_factory);
~ReadingListDownloadService() override;
@@ -111,6 +107,7 @@ class ReadingListDownloadService
bool had_connection_;
std::unique_ptr<reading_list::ReadingListDistillerPageFactory>
distiller_page_factory_;
+ std::unique_ptr<dom_distiller::DistillerFactory> distiller_factory_;
base::WeakPtrFactory<ReadingListDownloadService> weak_ptr_factory_;

Powered by Google App Engine
This is Rietveld 408576698