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

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

Issue 2646993002: Store distilled URL during distillation in Reading List on iOS (Closed)
Patch Set: 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/url_downloader.h
diff --git a/ios/chrome/browser/reading_list/url_downloader.h b/ios/chrome/browser/reading_list/url_downloader.h
index 3ed0dc7426311a806220b7043c1782a18a9cfb2c..76a404ec156d9e147ec4085a04cb7a14d13052b9 100644
--- a/ios/chrome/browser/reading_list/url_downloader.h
+++ b/ios/chrome/browser/reading_list/url_downloader.h
@@ -54,6 +54,7 @@ class URLDownloader {
// The path to downloaded file and title should not be used in case of
// failure.
using DownloadCompletion = base::Callback<void(const GURL&,
+ const GURL&,
SuccessState,
const base::FilePath&,
const std::string&)>;
@@ -131,6 +132,10 @@ class URLDownloader {
images,
const std::string& title);
+ // A callback called if the URL passed to the distilled leaded to a
jif 2017/01/20 10:23:19 needs verification: s/leaded/led/
Olivier 2017/01/20 10:47:07 Done.
+ // redirection.
+ void RedirectionCallback(const GURL& page_url, const GURL& redirected_url);
+
dom_distiller::DomDistillerService* distiller_service_;
reading_list::ReadingListDistillerPageFactory* distiller_page_factory_;
PrefService* pref_service_;
@@ -139,6 +144,7 @@ class URLDownloader {
std::deque<Task> tasks_;
bool working_;
base::FilePath base_directory_;
+ GURL distilled_url_;
std::unique_ptr<dom_distiller::DistillerViewerInterface> distiller_;
base::CancelableTaskTracker task_tracker_;

Powered by Google App Engine
This is Rietveld 408576698