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

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

Issue 2365353004: Add Content-Type header to net::ReportSender reports (Closed)
Patch Set: eroman comment 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..b6a8c0c89cffa03d1f9a905dfdbe2777aefe28d0 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& report) {
+ base::StringPiece content_type,
+ base::StringPiece report) {
latest_report_uri_ = report_uri;
- latest_report_ = report;
+ report.CopyToString(&latest_report_);
+ content_type.CopyToString(&latest_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;
« no previous file with comments | « chrome/browser/safe_browsing/mock_permission_report_sender.h ('k') | chrome/browser/safe_browsing/permission_reporter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698