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

Unified Diff: chrome/browser/lifetime/browser_close_manager_browsertest.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/lifetime/browser_close_manager_browsertest.cc
diff --git a/chrome/browser/lifetime/browser_close_manager_browsertest.cc b/chrome/browser/lifetime/browser_close_manager_browsertest.cc
index 5016975f28aac66cc2100356702899f7b700315f..c39bf244135a542c5df3cdd616a63569255d6587 100644
--- a/chrome/browser/lifetime/browser_close_manager_browsertest.cc
+++ b/chrome/browser/lifetime/browser_close_manager_browsertest.cc
@@ -174,16 +174,14 @@ class TestDownloadManagerDelegate : public ChromeDownloadManagerDelegate {
item, dangerous_callback);
}
- static void SetDangerous(
- const content::DownloadTargetCallback& callback,
- const base::FilePath& target_path,
- content::DownloadItem::TargetDisposition disp,
- content::DownloadDangerType danger_type,
- const base::FilePath& intermediate_path) {
- callback.Run(target_path,
- disp,
- content::DOWNLOAD_DANGER_TYPE_DANGEROUS_URL,
- intermediate_path);
+ static void SetDangerous(const content::DownloadTargetCallback& callback,
+ const base::FilePath& target_path,
+ content::DownloadItem::TargetDisposition disp,
+ content::DownloadDangerType danger_type,
+ const base::FilePath& intermediate_path,
+ content::DownloadInterruptReason interrupt_reason) {
+ callback.Run(target_path, disp, content::DOWNLOAD_DANGER_TYPE_DANGEROUS_URL,
+ intermediate_path, interrupt_reason);
}
};

Powered by Google App Engine
This is Rietveld 408576698