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

Unified Diff: net/http/transport_security_state_unittest.cc

Issue 2120683002: Fix up certificate error reporting histograms (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: components/BUILD.gn fix, don't build unit tests on iOS 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: net/http/transport_security_state_unittest.cc
diff --git a/net/http/transport_security_state_unittest.cc b/net/http/transport_security_state_unittest.cc
index 62deef2fc56662c7bce4019e94277df89783da6a..1fa689fe0d2c37635ffbbcd10e1b8dff80c96f33 100644
--- a/net/http/transport_security_state_unittest.cc
+++ b/net/http/transport_security_state_unittest.cc
@@ -1387,7 +1387,7 @@ TEST_F(TransportSecurityStateTest, UMAOnHPKPReportingFailure) {
// Check that the UMA histogram was updated when the report failed to
// send.
histograms.ExpectTotalCount(histogram_name, 1);
- histograms.ExpectBucketCount(histogram_name, mock_report_sender.net_error(),
+ histograms.ExpectBucketCount(histogram_name, -mock_report_sender.net_error(),
1);
}

Powered by Google App Engine
This is Rietveld 408576698