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

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

Issue 2365353004: Add Content-Type header to net::ReportSender reports (Closed)
Patch Set: make content type a required parameter to Send() Created 4 years, 3 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/mock_permission_report_sender.cc
diff --git a/chrome/browser/safe_browsing/mock_permission_report_sender.cc b/chrome/browser/safe_browsing/mock_permission_report_sender.cc
index dc48a0a12c03e9aa82bbc15c9364940f73d54877..e215e02aeb25ac8af970fd119e874ee1a3b6a59b 100644
--- a/chrome/browser/safe_browsing/mock_permission_report_sender.cc
+++ b/chrome/browser/safe_browsing/mock_permission_report_sender.cc
@@ -18,9 +18,11 @@ MockPermissionReportSender::~MockPermissionReportSender() {
}
void MockPermissionReportSender::Send(const GURL& report_uri,
+ const std::string& content_type,
const std::string& report) {
latest_report_uri_ = report_uri;
latest_report_ = report;
+ latest_content_type_ = content_type;
number_of_reports_++;
// BrowserThreads aren't initialized in the unittest, so don't post tasks
@@ -56,6 +58,10 @@ const std::string& MockPermissionReportSender::latest_report() {
return latest_report_;
}
+const std::string& MockPermissionReportSender::latest_content_type() {
+ return latest_content_type_;
+}
+
int MockPermissionReportSender::GetAndResetNumberOfReportsSent() {
int new_reports = number_of_reports_;
number_of_reports_ = 0;

Powered by Google App Engine
This is Rietveld 408576698