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

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 3 years, 9 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..353705472e93966201b8c8dc96443b47c37d2e77 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 ConfirmationCallback& callback) override {
file_chooser_displayed_ = true;
base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE,
- base::Bind(callback, (file_chooser_enabled_ ? suggested_path
- : base::FilePath())));
+ FROM_HERE, base::Bind(callback,
+ (file_chooser_enabled_
+ ? DownloadConfirmationResult::CONFIRMED
+ : DownloadConfirmationResult::CANCELED),
+ suggested_path));
}
void OpenDownload(content::DownloadItem* item) override {}
« no previous file with comments | « chrome/browser/download/download_target_info.cc ('k') | chrome/browser/lifetime/browser_close_manager_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698