Remove histograms for Reporting header parsing outcome
Some of these were expired. They were not very useful or actionable
anyway.
Bug: 975037, 1089017
Change-Id: I73e221e7fb1ba2106afade5b18a002f459bf009e
Fixed: 975037
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2265399
Reviewed-by: Charlie Harrison <[email protected]>
Reviewed-by: Eric Orth <[email protected]>
Commit-Queue: Lily Chen <[email protected]>
Cr-Commit-Position: refs/heads/master@{#782466}
diff --git a/net/http/http_network_transaction_unittest.cc b/net/http/http_network_transaction_unittest.cc
index 957a6b0..1aa55ea 100644
--- a/net/http/http_network_transaction_unittest.cc
+++ b/net/http/http_network_transaction_unittest.cc
@@ -19439,21 +19439,15 @@
TEST_F(HttpNetworkTransactionReportingTest,
DontProcessReportToHeaderNoService) {
- base::HistogramTester histograms;
clear_reporting_service();
RequestPolicy();
- histograms.ExpectBucketCount(
- ReportingHeaderParser::kHeaderOutcomeHistogram,
- ReportingHeaderParser::HeaderOutcome::DISCARDED_NO_REPORTING_SERVICE, 1);
+ // No crash.
}
TEST_F(HttpNetworkTransactionReportingTest, DontProcessReportToHeaderHttp) {
- base::HistogramTester histograms;
url_ = "http://www.example.org/";
RequestPolicy();
- histograms.ExpectBucketCount(
- ReportingHeaderParser::kHeaderOutcomeHistogram,
- ReportingHeaderParser::HeaderOutcome::DISCARDED_INVALID_SSL_INFO, 1);
+ EXPECT_EQ(0u, reporting_context()->cache()->GetEndpointCount());
}
TEST_F(HttpNetworkTransactionReportingTest, ProcessReportToHeaderHttps) {
@@ -19470,12 +19464,9 @@
TEST_F(HttpNetworkTransactionReportingTest,
DontProcessReportToHeaderInvalidHttps) {
- base::HistogramTester histograms;
CertStatus cert_status = CERT_STATUS_COMMON_NAME_INVALID;
RequestPolicy(cert_status);
- histograms.ExpectBucketCount(
- ReportingHeaderParser::kHeaderOutcomeHistogram,
- ReportingHeaderParser::HeaderOutcome::DISCARDED_CERT_STATUS_ERROR, 1);
+ EXPECT_EQ(0u, reporting_context()->cache()->GetEndpointCount());
}
#endif // BUILDFLAG(ENABLE_REPORTING)