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

Unified Diff: chrome/browser/safe_browsing/permission_reporter.cc

Issue 2648713002: Add response code to the success callback of ReportSender (Closed)
Patch Set: eroman and estark comments Created 3 years, 8 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/permission_reporter.cc
diff --git a/chrome/browser/safe_browsing/permission_reporter.cc b/chrome/browser/safe_browsing/permission_reporter.cc
index ba00b2fdd80a61e68b07177db25b2439920c962d..c4be17956f74e3ad950fd20a6f2f159814bebb35 100644
--- a/chrome/browser/safe_browsing/permission_reporter.cc
+++ b/chrome/browser/safe_browsing/permission_reporter.cc
@@ -164,10 +164,10 @@ void PermissionReporter::SendReport(const PermissionReportInfo& report_info) {
std::string serialized_report;
BuildReport(report_info, &serialized_report);
- permission_report_sender_->Send(GURL(kPermissionActionReportingUploadUrl),
- "application/octet-stream", serialized_report,
- base::Closure(),
- base::Callback<void(const GURL&, int)>());
+ permission_report_sender_->Send(
+ GURL(kPermissionActionReportingUploadUrl), "application/octet-stream",
+ serialized_report, base::Callback<void()>(),
+ base::Callback<void(const GURL&, int, int)>());
}
// static
« no previous file with comments | « chrome/browser/safe_browsing/notification_image_reporter.cc ('k') | chrome/browser/ssl/chrome_expect_ct_reporter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698