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

Unified Diff: chrome/browser/lifetime/browser_close_manager_browsertest.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/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 9500cc25c466aee834c260b7c02b88705ff393b7..9c7ff2df69fd2bb5925ff1f73884ec97e4a23122 100644
--- a/chrome/browser/lifetime/browser_close_manager_browsertest.cc
+++ b/chrome/browser/lifetime/browser_close_manager_browsertest.cc
@@ -211,16 +211,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);
}
};
« no previous file with comments | « chrome/browser/download/download_test_file_activity_observer.cc ('k') | chromecast/browser/cast_download_manager_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698