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

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

Issue 2491383002: Use Distilled path instead of DistilledURL. (Closed)
Patch Set: fix compilation Created 4 years, 1 month 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 c6a2a0bb8ecd6188e6f3bc84c3991ae6cc354ad4..fcd385128e65480e6f957abe59b51327a6e11042 100644
--- a/ios/chrome/browser/reading_list/reading_list_download_service.cc
+++ b/ios/chrome/browser/reading_list/reading_list_download_service.cc
@@ -177,13 +177,13 @@ void ReadingListDownloadService::RemoveDownloadedEntry(
void ReadingListDownloadService::OnDownloadEnd(
const GURL& url,
URLDownloader::SuccessState success,
- const GURL& distilled_url,
+ const base::FilePath& distilled_path,
const std::string& title) {
DCHECK(reading_list_model_->loaded());
if ((success == URLDownloader::DOWNLOAD_SUCCESS ||
success == URLDownloader::DOWNLOAD_EXISTS) &&
- distilled_url.is_valid()) {
- reading_list_model_->SetEntryDistilledURL(url, distilled_url);
+ !distilled_path.empty()) {
+ reading_list_model_->SetEntryDistilledPath(url, distilled_path);
} else if (success == URLDownloader::ERROR_RETRY) {
reading_list_model_->SetEntryDistilledState(url,
« no previous file with comments | « ios/chrome/browser/reading_list/reading_list_download_service.h ('k') | ios/chrome/browser/reading_list/reading_list_entry.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698