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

Unified Diff: chrome/browser/download/download_test_file_activity_observer.cc

Issue 2453633006: [downloads] Move platform specific code out of DownloadTargetDeterminer. (Closed)
Patch Set: . Created 4 years, 2 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: chrome/browser/download/download_test_file_activity_observer.cc
diff --git a/chrome/browser/download/download_test_file_activity_observer.cc b/chrome/browser/download/download_test_file_activity_observer.cc
index 5039abfbaee2f5c5ee548891b625b2e988dbfdee..642866b396fba55107a7c41f2818c4f729d39c06 100644
--- a/chrome/browser/download/download_test_file_activity_observer.cc
+++ b/chrome/browser/download/download_test_file_activity_observer.cc
@@ -52,15 +52,17 @@ class DownloadTestFileActivityObserver::MockDownloadManagerDelegate
}
protected:
- void PromptUserForDownloadPath(
- content::DownloadItem* item,
- const base::FilePath& suggested_path,
- const FileSelectedCallback& callback) override {
+ void RequestConfirmation(content::DownloadItem* item,
+ const base::FilePath& suggested_path,
+ DownloadConfirmationReason reason,
+ const FileSelectedCallback& callback) override {
file_chooser_displayed_ = true;
base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE,
- base::Bind(callback, (file_chooser_enabled_ ? suggested_path
- : base::FilePath())));
+ base::Bind(callback, (file_chooser_enabled_
+ ? DownloadConfirmationResult::CONFIRMED
+ : DownloadConfirmationResult::CANCELED),
+ suggested_path));
}
void OpenDownload(content::DownloadItem* item) override {}

Powered by Google App Engine
This is Rietveld 408576698