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

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

Issue 2650593003: 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/reading_list_download_service.cc
diff --git a/ios/chrome/browser/reading_list/reading_list_download_service.cc b/ios/chrome/browser/reading_list/reading_list_download_service.cc
index 6ae9d63f1fbaff627f932bce4ba4842a1c30027d..daab65cdce7a03cd09df80d8eff9eaa089d2c2f3 100644
--- a/ios/chrome/browser/reading_list/reading_list_download_service.cc
+++ b/ios/chrome/browser/reading_list/reading_list_download_service.cc
@@ -185,6 +185,7 @@ void ReadingListDownloadService::RemoveDownloadedEntry(const GURL& url) {
void ReadingListDownloadService::OnDownloadEnd(
const GURL& url,
+ const GURL& distilled_url,
URLDownloader::SuccessState success,
const base::FilePath& distilled_path,
const std::string& title) {
@@ -192,7 +193,8 @@ void ReadingListDownloadService::OnDownloadEnd(
if ((success == URLDownloader::DOWNLOAD_SUCCESS ||
success == URLDownloader::DOWNLOAD_EXISTS) &&
!distilled_path.empty()) {
- reading_list_model_->SetEntryDistilledInfo(url, distilled_path, url);
+ reading_list_model_->SetEntryDistilledInfo(url, distilled_path,
+ distilled_url);
if (!title.empty())
reading_list_model_->SetEntryTitle(url, title);
« no previous file with comments | « ios/chrome/browser/reading_list/reading_list_download_service.h ('k') | ios/chrome/browser/reading_list/url_downloader.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698