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

Unified Diff: net/http/transport_security_state.cc

Issue 2648713002: Add response code to the success callback of ReportSender (Closed)
Patch Set: Remove response code from error callback 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: net/http/transport_security_state.cc
diff --git a/net/http/transport_security_state.cc b/net/http/transport_security_state.cc
index a77562413ec7155cd9ad24c7f20249eeefcbe4f4..fb37f32229f2d0ccd6c5cbf425e6e34301af28f4 100644
--- a/net/http/transport_security_state.cc
+++ b/net/http/transport_security_state.cc
@@ -839,7 +839,7 @@ void TransportSecurityState::CheckExpectStaple(
}
report_sender_->Send(expect_staple_state.report_uri,
"application/json; charset=utf-8", serialized_report,
- base::Closure(),
+ base::Callback<void(int)>(),
base::Bind(RecordUMAForHPKPReportFailure));
}
@@ -1141,7 +1141,7 @@ TransportSecurityState::CheckPinsAndMaybeSendReport(
base::TimeDelta::FromMinutes(kTimeToRememberHPKPReportsMins));
report_sender_->Send(pkp_state.report_uri, "application/json; charset=utf-8",
- serialized_report, base::Closure(),
+ serialized_report, base::Callback<void(int)>(),
base::Bind(RecordUMAForHPKPReportFailure));
return PKPStatus::VIOLATED;
}

Powered by Google App Engine
This is Rietveld 408576698