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

Unified Diff: chrome/browser/safe_browsing/download_protection_service.h

Issue 2029903002: Add token field to ClientSafeBrowsingReportReqeust (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: tweak comments Created 4 years, 6 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/safe_browsing/download_protection_service.h
diff --git a/chrome/browser/safe_browsing/download_protection_service.h b/chrome/browser/safe_browsing/download_protection_service.h
index fffc858f19bd9b9fe8843acd6b4cd34f38708822..5c6ffbdc68e0ac7abb917605a501814ce8a88334 100644
--- a/chrome/browser/safe_browsing/download_protection_service.h
+++ b/chrome/browser/safe_browsing/download_protection_service.h
@@ -58,6 +58,10 @@ class DownloadProtectionService {
// Callback type which is invoked once the download request is done.
typedef base::Callback<void(DownloadCheckResult)> CheckDownloadCallback;
+ // Callback type which is invoked once the download content check is done.
+ typedef base::Callback<void(DownloadCheckResult, const std::string& token)>
+ CheckDownloadContentCallback;
+
// A type of callback run on the main thread when a ClientDownloadRequest has
// been formed for a download, or when one has not been formed for a supported
// download.
@@ -92,8 +96,9 @@ class DownloadProtectionService {
// method must be called on the UI thread, and the callback will also be
// invoked on the UI thread. This method must be called once the download
// is finished and written to disk.
- virtual void CheckClientDownload(content::DownloadItem* item,
- const CheckDownloadCallback& callback);
+ virtual void CheckClientDownload(
+ content::DownloadItem* item,
+ const CheckDownloadContentCallback& callback);
// Checks whether any of the URLs in the redirect chain of the
// download match the SafeBrowsing bad binary URL list. The result is
@@ -112,7 +117,7 @@ class DownloadProtectionService {
const GURL& requestor_url,
const base::FilePath& default_file_path,
const std::vector<base::FilePath::StringType>& alternate_extensions,
- const CheckDownloadCallback& callback);
+ const CheckDownloadContentCallback& callback);
// Display more information to the user regarding the download specified by
// |info|. This method is invoked when the user requests more information

Powered by Google App Engine
This is Rietveld 408576698